Finish including #61: the authority for the count, and where it is findable (1.28.2) - #94
Merged
Merged
Conversation
…ndable Moving #61 into REC-02 carried its substance and dropped two things worth keeping. The first is a phrase: the original said to use the release assets as the authority for the count. That matters, because a download figure can come from several places and they disagree — a package registry counts installs, a mirror counts its own traffic, and neither is what a release badge claims to show. The second is placement. #61 put this beside the badge rules, where somebody deciding what badges to show is already reading. REC-02 put it in the recommendations table, where somebody has to already know it exists. Status Badges now points at it and says plainly that it adds no condition to P08 and that its absence is not a gap, which is the question a reader of that section would otherwise have to go and answer elsewhere. Patch: a recommendation produces no result, and a cross-reference changes none either. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: f34e4f0b-ebd2-4432-9c55-9851a5e58c50
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#93 moved #61 into
REC-02and carried its substance. Re-reading the original diff against what landed, two things did not survive the move.The authority for the count. #61 said to "use the release assets as the authority". That matters: a download figure can come from several places and they disagree — a package registry counts installs, a mirror counts its own traffic, and neither is what a release badge claims to show.
REC-02now says it.Where it is findable. #61 put this beside the badge rules, where somebody deciding what badges to show is already reading.
REC-02put it in the recommendations table, where a reader has to already know it exists. Status Badges now points at it and answers the question that section would otherwise raise:Version
Patch. A recommendation produces no result, and a cross-reference changes none either.
Validation
standard.py --check, conformance.py --check, links.py, 159 tests, markdownlint — all green.