Skip to content

feat(mobile-app): Add extra experimental logging to the mobile-app subcommand#2636

Merged
szokeasaurusrex merged 4 commits intomasterfrom
ryan/add_extra_experimental_logging_to_the_mobile-app_subcommand
Jul 22, 2025
Merged

feat(mobile-app): Add extra experimental logging to the mobile-app subcommand#2636
szokeasaurusrex merged 4 commits intomasterfrom
ryan/add_extra_experimental_logging_to_the_mobile-app_subcommand

Conversation

@rbro112
Copy link
Copy Markdown
Member

@rbro112 rbro112 commented Jul 21, 2025

Per request, add an extra debug log to all invocations of the mobile-app subcommand.

@rbro112 rbro112 requested a review from szokeasaurusrex as a code owner July 21, 2025 22:22
Copy link
Copy Markdown
Member Author

rbro112 commented Jul 21, 2025

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

@rbro112 rbro112 changed the title Add extra experimental logging to the mobile-app subcommand feat(mobile-app): Add extra experimental logging to the mobile-app subcommand Jul 21, 2025
Comment thread src/commands/mobile_app/mod.rs Outdated
}

pub fn execute(matches: &ArgMatches) -> Result<()> {
debug!("EXPERIMENTAL: The mobile-app subcommand is experimental and may be updated without notice in the future.");
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Debug logs are not shown by default, as the default log level is WARN.

I will update this call to a warning! call.

…rimental_logging_to_the_mobile-app_subcommand
@szokeasaurusrex szokeasaurusrex enabled auto-merge (squash) July 22, 2025 08:59
@szokeasaurusrex szokeasaurusrex merged commit fcc6506 into master Jul 22, 2025
27 checks passed
@szokeasaurusrex szokeasaurusrex deleted the ryan/add_extra_experimental_logging_to_the_mobile-app_subcommand branch July 22, 2025 09:11
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.

2 participants