feat: add progress bar for long-running CLI operations#48
Conversation
feat: add reusable progress bar utility
Hell1213
left a comment
There was a problem hiding this comment.
Review: progress bar for CLI operations
Hi @geetanunakani, the progress bar utility is clean and reusable. Tests pass (72/72), ruff checks clean.
Conflict notice: PR #52 by @Jiya3177 also modifies src/oss_dev/cli/app.py (adds --json flag with early returns on the same functions). Both change the bodies of discover_repos, discover_issues, issues_list, analyze, and doctor. The merged result should wrap real work in both the progress bar AND support --json output. Consider coordinating the merge order.
Note: The time.sleep(2) placeholders are fine for stubs. When real implementations land, they should replace the sleep calls.
Hell1213
left a comment
There was a problem hiding this comment.
Progress bar utility is clean and reusable. The context manager pattern works well for this.
Summary
Related Issue
Fixes #ISSUE_NUMBER
Type of Change
Testing
uv run ruff checkpassesuv run mypypassesuv run pytestpassesDescription
Closes #32
Changes Made
progress_barutility insrc/oss_dev/cli/progress.pydiscover repos,discover issues, andanalyzecommandsdoctorcommandSkills Used