Skip to content

Commit 56c0024

Browse files
fix(proguard): Deprecate no-op --platform option (#2863)
### Description This option has been a no-op for some time; it likely should have been deprecated when we stopped reading it. ### Issues - Fixes #2850 - Fixes CLI-192
1 parent e6428dc commit 56c0024

File tree

3 files changed

+17
-4
lines changed

3 files changed

+17
-4
lines changed

CHANGELOG.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,11 @@
11
# Changelog
22

3+
## Unreleased
4+
5+
### Deprecations
6+
7+
- Deprecated the `upload-proguard` subcommand's `--platform` flag ([#2863](https://github.com/getsentry/sentry-cli/pull/2863)). This flag appears to have been a no-op for some time, so we will remove it in the next major.
8+
39
## 2.57.0
410

511
### New Features

src/commands/upload_proguard.rs

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -85,12 +85,13 @@ pub fn make_command(command: Command) -> Command {
8585
)
8686
.arg(
8787
Arg::new("platform")
88+
.hide(true)
8889
.long("platform")
8990
.value_name("PLATFORM")
9091
.requires("app_id")
9192
.help(
92-
"Optionally defines the platform for the app association. \
93-
[defaults to 'android']",
93+
"[DEPRECATED] This flag is a no-op, scheduled \
94+
for removal in Sentry CLI 3.0.0.",
9495
),
9596
)
9697
.arg(
@@ -145,6 +146,14 @@ pub fn make_command(command: Command) -> Command {
145146
}
146147

147148
pub fn execute(matches: &ArgMatches) -> Result<()> {
149+
if matches.get_one::<String>("platform").is_some() {
150+
log::warn!(
151+
"[DEPRECATION NOTICE] The --platform argument is deprecated, \
152+
and is scheduled for removal in Sentry CLI 3.0.0. \
153+
The argument is a no-op."
154+
);
155+
}
156+
148157
let paths: Vec<_> = match matches.get_many::<String>("paths") {
149158
Some(paths) => paths.collect(),
150159
None => {

tests/integration/_cases/upload_proguard/upload_proguard-help.trycmd

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,6 @@ Options:
1616
--auth-token <AUTH_TOKEN> Use the given Sentry auth token.
1717
--log-level <LOG_LEVEL> Set the log output verbosity. [possible values: trace, debug, info,
1818
warn, error]
19-
--platform <PLATFORM> Optionally defines the platform for the app association. [defaults
20-
to 'android']
2119
--quiet Do not print any output while preserving correct exit code. This
2220
flag is currently implemented only for selected subcommands.
2321
[aliases: --silent]

0 commit comments

Comments
 (0)