Skip to content

Commit fcc6506

Browse files
feat(mobile-app): Add extra experimental logging to the mobile-app subcommand (#2636)
Per request, add an extra debug log to all invocations of the `mobile-app` subcommand. --------- Co-authored-by: Daniel Szoke <daniel.szoke@sentry.io>
1 parent 1e67c3f commit fcc6506

File tree

5 files changed

+10
-0
lines changed

5 files changed

+10
-0
lines changed

src/commands/mobile_app/mod.rs

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,12 @@ pub fn make_command(mut command: Command) -> Command {
3535
}
3636

3737
pub fn execute(matches: &ArgMatches) -> Result<()> {
38+
log::warn!(
39+
"EXPERIMENTAL: The mobile-app subcommand is experimental. \
40+
The command is subject to breaking changes and may be removed \
41+
without notice in any release."
42+
);
43+
3844
macro_rules! execute_subcommand {
3945
($name:ident) => {{
4046
if let Some(sub_matches) =

tests/integration/_cases/mobile_app/mobile_app-upload-apk-all-uploaded.trycmd

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
```
22
$ sentry-cli mobile-app upload tests/integration/_fixtures/mobile_app/apk.apk
33
? success
4+
[..]WARN[..]EXPERIMENTAL: The mobile-app subcommand is experimental. The command is subject to breaking changes and may be removed without notice in any release.
45
Nothing to upload, all files are on the server
56
Successfully uploaded 1 file to Sentry
67
- tests/integration/_fixtures/mobile_app/apk.apk

tests/integration/_cases/mobile_app/mobile_app-upload-apk-no-token.trycmd

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
```
22
$ sentry-cli mobile-app upload tests/integration/_fixtures/mobile_app/apk.apk
33
? failed
4+
[..]WARN[..]EXPERIMENTAL: The mobile-app subcommand is experimental. The command is subject to breaking changes and may be removed without notice in any release.
45
error: Auth token is required for this request. Please run `sentry-cli login` and try again!
56

67
Add --log-level=[info|debug] or export SENTRY_LOG_LEVEL=[info|debug] to see more output.

tests/integration/_cases/mobile_app/mobile_app-upload-apk.trycmd

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
```
22
$ sentry-cli mobile-app upload tests/integration/_fixtures/mobile_app/apk.apk --sha test_sha
33
? success
4+
[..]WARN[..]EXPERIMENTAL: The mobile-app subcommand is experimental. The command is subject to breaking changes and may be removed without notice in any release.
45
Successfully uploaded 1 file to Sentry
56
- tests/integration/_fixtures/mobile_app/apk.apk
67

tests/integration/_cases/mobile_app/mobile_app-upload-ipa.trycmd

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
```
22
$ sentry-cli mobile-app upload tests/integration/_fixtures/mobile_app/ipa.ipa --sha test_sha
33
? success
4+
[..]WARN[..]EXPERIMENTAL: The mobile-app subcommand is experimental. The command is subject to breaking changes and may be removed without notice in any release.
45
Successfully uploaded 1 file to Sentry
56
- tests/integration/_fixtures/mobile_app/ipa.ipa
67

0 commit comments

Comments
 (0)