Skip to content

#680: make the printed lines add up to the total - #706

Open
morphqdd wants to merge 1 commit into
zerocracy:masterfrom
morphqdd:680
Open

#680: make the printed lines add up to the total#706
morphqdd wants to merge 1 commit into
zerocracy:masterfrom
morphqdd:680

Conversation

@morphqdd

@morphqdd morphqdd commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

points rounded the sum of the raw values while greeting formatted each line
with %+d, which truncates, so the itemised list and the headline number
disagreed. On the shipped bylaws this happened for a large share of the inputs.

Each line is rounded now, and the total is the sum of those rounded lines, so the
numbers a contributor is shown add up to what they are told they earned.

Closes #680

yegor256 commented Sep 3, 2026

Copy link
Copy Markdown
Member

Thanks — makes the itemised lines and the total round the same way. Small, focused, tested. CI is green.

@rultor merge


Generated by Claude Code

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.

Fbe::Award::Bill#greeting prints line values that do not add up to the total it states

2 participants