Skip to content

Add a code review skill (.github/skills/code-review/SKILL.md) - #15171

Merged
cclauss merged 1 commit into
TheAlgorithms:masterfrom
priya-sundaram-dev:code-review-skill
Sep 3, 2026
Merged

Add a code review skill (.github/skills/code-review/SKILL.md)#15171
cclauss merged 1 commit into
TheAlgorithms:masterfrom
priya-sundaram-dev:code-review-skill

Conversation

@priya-sundaram-dev

@priya-sundaram-dev priya-sundaram-dev commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Fixes: #15167

As requested in #15167 — this is the separate PR for .github/skills/code-review/SKILL.md, expanding the draft into an executable skill.

Credit to @cclauss for the original draft in #15167; this builds on it with the suggestions from my review there.

What it does

Turns the review process into something any reviewer (human or AI) can run the same way every time:

  • Mirrors the CONTRIBUTING.md sections explicitlyBefore contributing / Is this an algorithm?, Coding Style, Other Requirements for Submissions — as checkable items so nothing is skipped.
  • Fixed output shape for the verdict (Is this an algorithm? / duplicate check / coding style / other requirements / verdict) so reviews are comparable and easy to automate later.
  • Tone rule: cite the exact rule and suggest the fix rather than a bare 'rejected' — keeps first-time and Hacktoberfest contributors coming back.
  • Maps findings to existing labels (require tests, require type hints, require descriptive names, tests are failing, awaiting reviews) so a review naturally produces the label the cleanup scripts key off.

Describe your change:

  • Add a skill?

Checklist:

  • I have read CONTRIBUTING.md.
  • This pull request is all my own work.
  • I know that pull requests will not be merged if they fail the automated tests.

@algorithms-keeper algorithms-keeper Bot added documentation This PR modified documentation files awaiting reviews This PR is ready to be reviewed labels Sep 3, 2026
@algorithms-keeper algorithms-keeper Bot removed the awaiting reviews This PR is ready to be reviewed label Sep 3, 2026
@cclauss
cclauss merged commit 6e0a4d4 into TheAlgorithms:master Sep 3, 2026
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation This PR modified documentation files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants