fix(build): Allow clearing string arguments to build upload

[szokeasaurusrex]

Description

Don't serialize empty-string values for --vcs-provider, --head-repo-name, --head-ref, --base-ref, and --base-repo-name into HTTP requests to the assemble build artifact endpoint. This allows users to set these values to empty strings to override any default values.

Issues

• Resolves Skip serializing certain fields for build upload #2941
• Resolves CLI-224

[szokeasaurusrex]

• fix(build): Allow clearing string arguments to build upload #2946 👈 (View in Graphite)
• ref(build): Add client-side validation for SHA fields #2945
• ref(build): Move VcsInfo next to other build upload API types #2944
• ref(build): Use VcsInfo directly in ChunkedBuildRequest #2943
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

[linear]

CLI-224 Skip serializing certain fields for `build upload`

[szokeasaurusrex]

@runningcode, I did not make it possible to clear --pr-number in this PR. Currently, attempting to clear it results in a validation error, as the argument is not configured in a way to allow empty values:

% cargo run -- build upload --pr-number ""
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.69s
     Running `target/debug/sentry-cli build upload --pr-number ''`
error: invalid value '' for '--pr-number <pr_number>': cannot parse integer from empty string

For more information, try '--help'.

[runningcode]

Thanks! I'm sure we'll get a bug report one day about clearing the PR number but we can fix it then. It's currently only automatically inferred in Github actions.