Skip to content

Conversation

@nicokh-sb
Copy link
Contributor

What does this PR do?

Adds RichConsoleReporter - an option to show progress in an interactive way in the console

Changes

  • Added RichConsoleReporter class
  • Updated API with the new reporter option

Related Issues

Fixes SparkBeyond/ao-core#158

@nicokh-sb nicokh-sb requested a review from danarmak December 21, 2025 10:21
@nicokh-sb nicokh-sb self-assigned this Dec 21, 2025
Copy link
Collaborator

@danarmak danarmak left a comment

Choose a reason for hiding this comment

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

This just shows a tree of static text. I thought we were going to display a progress bar at each node in the tree.

@nicokh-sb
Copy link
Contributor Author

nicokh-sb commented Dec 24, 2025

@danarmak Regarding the progress bars, I abandoned this idea because in my opinion too many progress bars at the same time will not look good UI-wise. The text isn't static, it updates the count/percentages. @yossihermush approved the look of it before, but of course we can add it if needed (or make it optional)

@nicokh-sb nicokh-sb force-pushed the feat/355-Add-Rich-Reporter branch from 69da4f4 to 455a233 Compare January 13, 2026 09:01
@danarmak danarmak merged commit 817ce83 into main Jan 13, 2026
1 check passed
@danarmak danarmak deleted the feat/355-Add-Rich-Reporter branch January 13, 2026 15:12
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.

3 participants