Skip to content
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

Refine numbers in headers of summary table #17

Merged
merged 2 commits into from
Jan 5, 2023
Merged

Refine numbers in headers of summary table #17

merged 2 commits into from
Jan 5, 2023

Conversation

jwodder
Copy link
Member

@jwodder jwodder commented Jan 5, 2023

Closes #15.

healthstatus.py Outdated Show resolved Hide resolved
Co-authored-by: Yaroslav Halchenko <debian@onerussian.com>
@yarikoptic
Copy link
Member

cool. Could you actually rerender that header stats so we would see the results -- all the information is there, right?

@jwodder
Copy link
Member Author

jwodder commented Jan 5, 2023

@yarikoptic The information is there in the status.yaml files, but the script isn't built for getting information back out of them. Unless you foresee the README repeatedly needing to be regenerated from the status.yaml files, I don't think updating the script to do this is worth it.

@yarikoptic
Copy link
Member

ok, it would be a separate development then anyways. Given that it takes days (or weeks) for a full pass ATM I am afraid we aren't to see the effect of this fix for awhile.

@yarikoptic yarikoptic merged commit bf5fb75 into main Jan 5, 2023
@yarikoptic yarikoptic deleted the gh-15 branch January 5, 2023 19:49
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.

total number in the header should not be just a sum
2 participants