Skip to content

Feat: add second level of verbosity#3982

Merged
treysp merged 3 commits intomainfrom
trey/verbosity
Mar 19, 2025
Merged

Feat: add second level of verbosity#3982
treysp merged 3 commits intomainfrom
trey/verbosity

Conversation

@treysp
Copy link
Contributor

@treysp treysp commented Mar 12, 2025

Current behavior

By default, the plan command CLI output:

  • Condenses long lists of indirectly modified models
  • Only lists individual models for the model evaluation progress bar

If the --verbose flag is passed the CLI output:

  • Lists all indirectly modified models
  • Lists individual models for both table creation and view promotion progress bars

New behavior

This PR introduces levels of verbosity, allowing us to unbundle list condensation from progress bar behavior. There are now three levels:

  • Default: current default behavior
  • -v or --verbose flag: list models for all progress bars (but condense indirect model lists)
  • -vv or two --verbose flags: display full indirectly modified lists, list models for all progress bars

@treysp treysp force-pushed the trey/verbosity branch 3 times, most recently from 1d981c0 to 625db38 Compare March 12, 2025 23:25
@treysp treysp marked this pull request as ready for review March 12, 2025 23:25
@treysp treysp force-pushed the trey/verbosity branch 2 times, most recently from f2b9b87 to 032d079 Compare March 13, 2025 19:44
@treysp treysp enabled auto-merge (squash) March 19, 2025 17:33
@treysp treysp merged commit 61ff15a into main Mar 19, 2025
21 of 22 checks passed
@treysp treysp deleted the trey/verbosity branch March 19, 2025 17:37
treysp added a commit that referenced this pull request Mar 26, 2025
treysp added a commit that referenced this pull request Mar 26, 2025
treysp added a commit that referenced this pull request Mar 26, 2025
treysp added a commit that referenced this pull request Mar 27, 2025
treysp added a commit that referenced this pull request Mar 27, 2025
treysp added a commit that referenced this pull request Mar 27, 2025
treysp added a commit that referenced this pull request Mar 28, 2025
treysp added a commit that referenced this pull request Mar 28, 2025
treysp added a commit that referenced this pull request Mar 31, 2025
treysp added a commit that referenced this pull request Mar 31, 2025
treysp added a commit that referenced this pull request Mar 31, 2025
afzaljasani pushed a commit that referenced this pull request Apr 8, 2025
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