Skip to content

Add --puby-verbose and hide noisy informational messages - #15

Merged
Rexios80 merged 3 commits into
masterfrom
feature/puby-verbose
Jul 23, 2026
Merged

Rexios80 merged 3 commits into
masterfrom
feature/puby-verbose

Conversation

@Rexios80

Copy link
Copy Markdown
Owner

Summary

  • Adds --puby-verbose for more detailed puby output (stripped from delegated command args, unlike --verbose)
  • Hides project skipping and engine override messages by default
  • Updates docs, help text, and tests that assert on skip messages

Test plan

  • Run puby get in a monorepo and confirm skip/override messages are hidden
  • Run puby get --puby-verbose and confirm those messages appear
  • Confirm puby get --verbose still forwards --verbose to pub/flutter
  • Confirm puby link still prints per-project resolve/download lines without --puby-verbose
  • Run the test suite (dart test)

@cursor

cursor Bot commented Jul 22, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR introduces a new --puby-verbose flag that enables additional puby-specific logging while keeping delegated command arguments clean, and it reduces default console noise by hiding engine override and project-skip informational messages unless explicitly requested.

Changes:

  • Add --puby-verbose parsing on Command and expose it via Command.verbose.
  • Gate engine override + project skipping/exclusion messages behind --puby-verbose.
  • Update help text, README, changelog, and adjust tests that assert on skip output.

Reviewed changes

Copilot reviewed 8 out of 11 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
lib/command.dart Parses --puby-verbose (stripped from delegated args) and exposes Command.verbose.
bin/projects.dart Hides engine override + skip/exclusion informational messages unless command.verbose is set.
bin/link.dart Hides “Skipping workspace member” output unless command.verbose is set.
bin/puby.dart Documents --puby-verbose in CLI help text.
README.md Adds a brief note documenting --puby-verbose.
CHANGELOG.md Adds a 1.39.0 entry describing the new flag and reduced default noise.
test/pub_test.dart Updates test invocations to include --puby-verbose where skip output is asserted.
test/link_test.dart Updates workspace-member skip-output test to include --puby-verbose.
test/run_test.dart Updates test invocation to include --puby-verbose for skip assertions.
test/gen_test.dart Updates test invocation to include --puby-verbose for skip assertions.
test/config_test.dart Updates test invocation to include --puby-verbose for skip assertions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread test/pub_test.dart Outdated
@Rexios80
Rexios80 merged commit 68b1269 into master Jul 23, 2026
2 of 3 checks passed
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