-
Notifications
You must be signed in to change notification settings - Fork 594
fix(cli): make --progress flag visible and use it in sync-to command #4877
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…umentation - Update progress flag description from "progress bar" to "progress output" for clarity - Document progress control features in Logging, Synchronization, and Command-Line reference - Support --no-progress flag for cleaner automation and scripting usage
There was a problem hiding this comment.
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 makes the --progress flag visible to users and ensures the sync-to command respects the global progress setting. Previously, the progress flag was hidden and the sync-to command didn't honor the user's progress preferences.
- Removed the
.Hidden()modifier from the--progressflag to make it visible in help output - Added progress control logic to the
sync-tocommand to respect the global progress setting - Updated documentation to explain the progress flag and its usage in automation scenarios
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| cli/cli_progress.go | Made the --progress flag visible by removing .Hidden() modifier |
| cli/command_repository_sync.go | Added progress control to sync-to command by checking progress state before output |
| site/content/docs/Reference/Command-Line/_index.md | Added global flags documentation including the --progress flag |
| site/content/docs/Advanced/Synchronization/_index.md | Added automation section explaining --no-progress usage for scripts |
| site/content/docs/Advanced/Logging/_index.md | Added progress output section explaining progress control separate from logging |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #4877 +/- ##
==========================================
+ Coverage 75.86% 76.78% +0.92%
==========================================
Files 470 535 +65
Lines 37301 40939 +3638
==========================================
+ Hits 28299 31436 +3137
- Misses 7071 7457 +386
- Partials 1931 2046 +115 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
This PR addresses #3542 by making the
--progressflag visible and ensuring thesync-tocommand respects the global progress setting.