Skip to content

feat: add --fix to contributor checker - #1217

Merged
yakew7 merged 1 commit into
yakew7:mainfrom
dxnnystyles:feat/issue-1201-contributor-fix
Oct 29, 2026
Merged

yakew7 merged 1 commit into
yakew7:mainfrom
dxnnystyles:feat/issue-1201-contributor-fix

Conversation

@dxnnystyles

Copy link
Copy Markdown
Contributor

Summary

this adds --fix to refresh known merged-PR counts and first-merge dates, then restore the existing contributor order while preserving commit counts and profile prose.

Type

  • Audit
  • Explainer
  • Bug fix
  • Other

Audit checklist

N/A — this is a repository tooling change, not an audit.

  • I opened or linked a corresponding issue first — N/A
  • The folder is named after the domain, not the dataset — N/A
  • unfair.py includes protected attributes and prints the required output format — N/A
  • fair.py removes protected attributes and identified proxy variables — N/A
  • Both scripts use random_state=42 and an 80/20 train/test split — N/A
  • Proxy variables were actually tested, not just guessed — N/A
  • unfair.png and fair.png are included as PNG screenshots — N/A
  • The dataset is public and accessible without login or payment — N/A
  • The dataset file is included, or DATA.md is included if the file is too large — N/A
  • README.md includes the new results row and audit section — N/A
  • A notebook was added if the audit benefits from one — N/A

Before fairness gap: N/A

After fairness gap: N/A

Reduction: N/A

Protected attribute(s): N/A

Proxy variables dropped: N/A


Explainer checklist

N/A — this change does not add an explainer.

  • The file is in explainers/ and uses lowercase hyphenated naming — N/A
  • It includes a plain-language definition — N/A
  • It uses a real example from this repo or a documented real-world case — N/A
  • It includes runnable Python detection or measurement code — N/A
  • It acknowledges limitations or trade-offs — N/A
  • It links to related explainers or repo projects — N/A
  • It includes 2-3 primary sources — N/A
  • The Explainers table in README.md was updated — N/A

Linked issue

Closes #1201

Verification

  • focused contributor-checker tests: 12 passed
  • make test-fast: 1,200 passed, 33 skipped, 27 deselected
  • make lint and git diff --check: passed, including the contributor-recognition check
  • slow tests and hosted CI were not run

@vercel

vercel Bot commented Oct 29, 2026

Copy link
Copy Markdown
Contributor

@dxnnystyles is attempting to deploy a commit to the yashkewlani2020-gmailcom's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

Copy link
Copy Markdown

@yakew7 @ahmdkaml @propcgamer20-png - new PR touching a path you own, please review.

@yakew7
yakew7 merged commit bd183ac into yakew7:main Oct 29, 2026
18 of 19 checks passed
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.

check_contributors.py should offer --fix to rewrite counts and order

2 participants