Skip to content

Add stats to progress bar#396

Merged
mrnugget merged 4 commits into
mainfrom
mrn/stats
Nov 26, 2020
Merged

Add stats to progress bar#396
mrnugget merged 4 commits into
mainfrom
mrn/stats

Conversation

@mrnugget

@mrnugget mrnugget commented Nov 25, 2020

Copy link
Copy Markdown
Contributor

@LawnGnome LawnGnome left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I know this is in draft, but this LGTM. A good win in terms of providing more detail without being confusing!

Comment thread cmd/src/campaign_progress_printer.go Outdated
@mrnugget mrnugget marked this pull request as ready for review November 26, 2020 12:11
@mrnugget mrnugget requested a review from a team November 26, 2020 12:11
var (
campaignsPendingColor = output.StylePending
campaignsSuccessColor = output.StyleSuccess
campaignsErrorColor = output.StyleWarning

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

we don't have errors anymore? 🤔

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It was unused 🤷

@mrnugget mrnugget merged commit 561674d into main Nov 26, 2020
@mrnugget mrnugget deleted the mrn/stats branch November 26, 2020 13:23
scjohns pushed a commit that referenced this pull request Apr 24, 2023
* Add stats to progress bar

* Add SetLabelAndRecalc

* Add changelog entry

* Print task execution time
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.

src-cli: Provide better stats/numbers while running a campaign

3 participants