Skip to content

Add automated PR check for contributing criteria#1776

Open
mre wants to merge 1 commit intomasterfrom
add-pr-check
Open

Add automated PR check for contributing criteria#1776
mre wants to merge 1 commit intomasterfrom
add-pr-check

Conversation

@mre
Copy link
Member

@mre mre commented Mar 20, 2026

Adds a Rust binary crate at data/pr-check and a new workflow .github/workflows/pr-check.yml that automatically checks every PR touching data/tools/ against the contributing criteria.

For each newly added tool YAML the check fetches data from the GitHub API and evaluates:

  • Stars (minimum 20)
  • Human contributors (minimum 2)
  • Repository age (minimum 3 months)

Results are posted as a single Markdown comment on the PR, updated in place on each push rather than adding a new comment every time. Non-GitHub sources are flagged for manual review. The CI step exits with code 1 when any criterion is not met, so the check turns red.

The workflow only runs when data/tools/ files change and caches the cargo build between runs.

Adds a new Rust binary crate at data/pr-check that runs on every PR
touching data/tools/ and checks each newly added tool YAML against
the contributing criteria:

- More than 20 stars
- More than one contributor
- Repository is at least 3 months old

The check posts a formatted Markdown comment on the PR summarising
the results per tool, updating the existing bot comment on re-push
rather than adding a new one each time. Non-GitHub sources are
flagged for manual review. The CI step exits with a non-zero code
when any criterion is not met, turning the check red.

The workflow only triggers on PRs that touch data/tools/ and caches
the cargo build to keep runs fast.
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.

1 participant