Extended labels.yml for auto-close nonsense PRs by 0saurabh0 · Pull Request #19613 · django/django · GitHub | Latest TMZ Celebrity News & Gossip | Watch TMZ Live
Skip to content

Extended labels.yml for auto-close nonsense PRs #19613

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

0saurabh0
Copy link
Contributor

This PR extends the labels.yml configuration used for labeling and managing PRs to help automatically identify and close low-quality or non-substantive contributions. These types of PRs often come from new contributors following the contributing tutorial, which can unintentionally encourage test PRs that do not provide meaningful changes to the codebase or documentation.

This change is part of my GSoC'25 project to automate and improve Django’s contribution workflow.

Checklist

  • This PR targets the main branch.
  • The commit message is written in past tense, mentions the ticket number, and ends with a period.
  • I have checked the "Has patch" ticket flag in the Trac system.
  • I have added or updated relevant tests.
  • I have added or updated relevant docs, including release notes if applicable.
  • I have attached screenshots in both light and dark modes for any UI changes.

@github-actions github-actions bot added the no ticket Based on PR title, no linked Trac ticket label Jul 1, 2025
@felixxm
Copy link
Member

felixxm commented Jul 2, 2025

Has this been discussed anywhere? I don't see much benefit in this, and closing such PRs is not a big enough burden, that would justify adding any script for this purpose. I'm strongly against it.

@0saurabh0
Copy link
Contributor Author

Has this been discussed anywhere?

I think so.
cc @sarahboyce @nessita @LilyAcorn

@felixxm
Copy link
Member

felixxm commented Jul 2, 2025

Has this been discussed anywhere?

I think so. cc @sarahboyce @nessita @LilyAcorn

I'm a bit confused. Why you pinged so many folks? If it was discussed and accepted it's enough to point me to this discussion.

@0saurabh0
Copy link
Contributor Author

https://docs.google.com/document/d/1cifdwoPGxOiSZa4QiibdqAdWJbjF3BL7YUm3uv61otM/edit?tab=t.0#heading=h.a45nsz1907yp
It doesn’t include details about this PR, as the work had already begun following a discussion with them.

@shangxiao
Copy link
Contributor

shangxiao commented Jul 2, 2025

I almost thought this was also a "nonsense PR", there's no ticket linked to in the description, no way of verifying legitimacy other than it mentions GSoC and was able to find this: https://summerofcode.withgoogle.com/programs/2025/projects/xF6r182s 😅

Anyway I agree with Felix... The total time spent creating, testing, reviewing and further maintaining this script seems like it would far outweigh the total time taken closing toast PRs.

If anything I'd recommend removing the toast bit from the docs completely and referring people to the official GitHub docs if they want to learn about PRs 🤔

@felixxm
Copy link
Member

felixxm commented Jul 2, 2025

Anyway I agree with Felix... The total time spent creating, testing, reviewing and further maintaining this script seems like it would far outweigh the total time taken closing toast PRs.

Exactly, we're talking about 5-10 (at most, e.g. 86 "toast" PRs in the last 7 years, ~12 per year) invalid PRs per month, that will take the same number of minutes to close. On the other hand we have 600 LOC PR that need to be understood, reviewed, merged and maintained. Furthermore, it will catch half of those cases and will likely mark at least twice as many valid PRs. I appreciate your work but this is an overkill to me.

@thibaudcolas
Copy link
Member

👋 here’s Saurabh and Lily’s high-level project description: Automate processes within Django contribution workflow

@shangxiao @felixxm I think we need to keep an open mind for stuff like this. There’s lots of room for more automation, and perhaps the ROI isn’t big enough for just this use case, but it seems like a fine one to experiment with? And I’d hope a triage script doesn’t require as much correctness as other 600 LOC PRs.

Re "legitimacy", if in doubt – assume the best.

@felixxm
Copy link
Member

felixxm commented Jul 3, 2025

👋 here’s Saurabh and Lily’s high-level project description: Automate processes within Django contribution workflow

@shangxiao @felixxm I think we need to keep an open mind for stuff like this. There’s lots of room for more automation, and perhaps the ROI isn’t big enough for just this use case, but it seems like a fine one to experiment with? And I’d hope a triage script doesn’t require as much correctness as other 600 LOC PRs.

Re "legitimacy", if in doubt – assume the best.

I still think we're spending too much time trying to automate instead of actually doing something.

This script will help to close ~1 "toast" PR per month, and in the same time mark dozens of PRs with test and tutorial as invalid that are not (we had thousand of such PRs and most of them were valid). It's debatable that this will improve contribution workflow, especially for contributors 🤷

@thibaudcolas
Copy link
Member

👍 fair enough. Again this is an in-progress GSoC project, so although the value is debatable in your mind, there’s plenty enough interest from others so important to give this a chance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no ticket Based on PR title, no linked Trac ticket
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants

TMZ Celebrity News – Breaking Stories, Videos & Gossip

Looking for the latest TMZ celebrity news? You've come to the right place. From shocking Hollywood scandals to exclusive videos, TMZ delivers it all in real time.

Whether it’s a red carpet slip-up, a viral paparazzi moment, or a legal drama involving your favorite stars, TMZ news is always first to break the story. Stay in the loop with daily updates, insider tips, and jaw-dropping photos.

🎥 Watch TMZ Live

TMZ Live brings you daily celebrity news and interviews straight from the TMZ newsroom. Don’t miss a beat—watch now and see what’s trending in Hollywood.