Skip to content

feat: add progress bar for long-running CLI operations#48

Merged
Hell1213 merged 2 commits into
Hell1213:mainfrom
geetanunakani:feature/progress-bar
Jun 1, 2026
Merged

feat: add progress bar for long-running CLI operations#48
Hell1213 merged 2 commits into
Hell1213:mainfrom
geetanunakani:feature/progress-bar

Conversation

@geetanunakani

Copy link
Copy Markdown
Contributor

Summary

Related Issue

Fixes #ISSUE_NUMBER

Type of Change

  • Bug fix
  • New feature
  • Documentation
  • Refactoring
  • Test changes
  • CI/Chore

Testing

  • uv run ruff check passes
  • uv run mypy passes
  • uv run pytest passes
  • Manual testing done

Description

Closes #32

Changes Made

  • Added reusable progress_bar utility in src/oss_dev/cli/progress.py
  • Integrated progress bar into discover repos, discover issues, and analyze commands
  • Also added to doctor command
  • Shows spinner, elapsed time, and current operation description using Rich library

Skills Used

  • Python, Rich library, CLI/UX design

@Hell1213 Hell1213 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

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 Hell1213 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Progress bar utility is clean and reusable. The context manager pattern works well for this.

@Hell1213 Hell1213 merged commit 2474490 into Hell1213:main Jun 1, 2026
0 of 2 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.

Implement progress bar for long-running operations

2 participants