chadtmiller.com/knotless
git that lives in your Atmosphere account
1
10
web: Badge cards describe what an account did, in plain words
Merged Approved
This is a copy-only change to web/src/lib/badges.ts. It rewrites the badge count lines in the active voice with "their", and it changes the tier line to "Bronze, tier 2 of 4." It also removes ORDINALS. I found no problems.
- Tier line:
METALS.lengthis 4, so "tier N of 4" matches the old wording. TheMETALS.indexOf(...) + 1expression is safe, becauseatcan only be one of the four metals on the tiers path. - Removed code: I searched for the old strings and for
ORDINALSand found no other users. That includes the Rust-side badges, which have no matching strings. - Pluralisation:
pulls(n)and the "it"/"they" switch are unchanged and still read correctly in the new sentences.
Stargazer is the one count line without a verb. It still reads fine.
All checks have passed
7 passed, 2 skipped
Merged as b98872f
What changes
Counts. Each badge's line on its card now leads with a verb and says "their" for the account: "Got 3 pulls merged into other accounts' repositories.", "Published 4 releases.", "Moved their account to a new PDS." Stargazer says it counts their most-starred repository, which "one repository" left unclear.
Tiers. "Bronze tier, the second of four." becomes "Bronze, tier 2 of 4." A badge earned once says "One-time badge." and First light says "Limited. Earned only before 1.0."
ORDINALSgoes, now unused.Why
The cards read like the design doc: third person ("It moved its account"), passive ("found by halving a batch that failed"), and the queue's internals ("turned away for a conflict"). A profile is read by people who never opened that doc.
How it was checked
Copy only, in
web/src/lib/badges.ts. No test asserts these strings. Badge kinds, counts and the lexicon are untouched. Left to CI.Shortcomings
Peacemaker, In good company, Smooth landing and Bisected still assume the reader knows what the merge queue is. If a shot opens a badge card, it moves.