Skip to content

Conversation

vlechemin
Copy link

@ceorourke ceorourke requested a review from a team January 3, 2022 17:54
@ceorourke
Copy link
Member

Hey @vlechemin I've put this in our backlog to review, but I can't guarantee when we'll get to it. Thanks for the submission!

|`set_commits`|Specify whether to set commits for the release. Either "auto" or "skip".|"auto"|
|`projects`|Space-separated list of paths of projects. When omitted, falls back to the environment variable `SENTRY_PROJECT` to determine the project.|-|
|`url_prefix`|Adds a prefix to source map urls after stripping them.|-|
|`strip_common_prefix`|Will remove a common prefix from uploaded filenames. Useful for removing a path that is build-machine-specific.|`false`|
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you replace these two lines with this:

|`strip_prefix`|Strips the given prefix from uploaded filenames. Useful for removing a path that is build-machine-specific.|-|
|`strip_common_prefix`|Similar to `strip-prefix` but strips the most common prefix on all sources references when set to `true`.|`false`|

This order matches the sentry-cli documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants