Cut down on Spammy-AI requests, find good new contributors - #2030
Merged
Merged
Conversation
Runs gh-profiler (Eric Matthes) against the author of a PR from anyone with no merged PR here yet, and comments the concise summary only when there is a red flag or a yellow on PR/issue activity. A yellow on the profile section alone means an account with sparse bio fields, which is not worth a public comment. The full profile always goes to the run log. pull_request_target is safe here: the job never checks out or runs PR code. Adds a PR template disclosing the check to first-time contributors.
The templates only prefill in the web UI, so issues and PRs opened through the API or `gh` have been arriving without them. State in CONTRIBUTING.md and AGENTS.md that they are to be read from `.github/` and followed either way. Also documents the first-PR profile check in both.
mscuthbert
changed the title
|
coverage: 93.348%. remained the same — contributor-profile into master |
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
2 participants
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.
This repo, like many, has seen a big rise in first-time contributors. Most are GREAT! and we're moving faster than in years. But many are slop contributions; drive-by "i hope this works" etc.
Add a check for new contributors to see if there are problems with accounts. Add disclosures that this will happen (since there doesn't seem to be an easy way to make the check results private). Add info about templates being required.