docs: add CONTRIBUTORS.md - #172
Merged
Merged
Conversation
Several of this project's most substantial contributions (design discussion in issue threads, caught errors, proposed schemas) don't appear in GitHub's commit-based contributor graph at all. This file credits that work by what was actually done, not a flat name list, avoiding the same flattening problem the researcher-field correction existed to prevent at the record level. Every claim independently verified against the real issues/PRs before writing anything down (#121, #94, #98, #160, #171, #68, #130, #125, alexgreensh/repo-forensics#39). One claim from the original draft (a specific ISO 27001 Annex A crosswalk proposal attributed to a Reddit user) had no trace anywhere in this repo, unlike everything else here, and was dropped rather than published unverified.
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.
Adds specific, honest credit for external contributions that predate or exceed what GitHub's automatic contributor graph would show. Organized by nature of contribution, not a flat list, so a single suggestion and a multi-month design collaboration don't read as equivalent.
Verification
Every claim checked against the real issue/PR threads before being written down, not taken from the drafting brief at face value:
source_commitfield without realizingcommit, which he'd written himself in Add the crosswalk schema every crosswalk already declares #121, already existed; confirmed word-for-word in the feat: state what commit pins, and let a side declare it cannot be pinned #171 thread).One claim from the original draft -- a specific ISO 27001 Annex A crosswalk proposal attributed to a Reddit user (VentureandCode_LEGAL) -- had no trace anywhere in this repo's issues or PRs, unlike every other claim here, which all resolved to a specific checkable thread. Rather than publish an unverified claim about a real person, that section was dropped. (The same person's real, verified contribution -- the confidence_baseline corroboration quoted directly in #98 -- is still credited via astrogilda's entry, since that one is checkable.)
Test plan
grep -c "—" CONTRIBUTORS.md README.md-- 0 new em-dashes in either filepython3 scripts/validate_records.py-- 77/77 valid (unaffected, no record changes)pytest tests/ -x -q-- 309 passed