Skip to content

CI: Add automation gating PR creation to assigned contributors#65849

Draft
rhshadrach wants to merge 2 commits into
pandas-dev:mainfrom
rhshadrach:pr_automation
Draft

CI: Add automation gating PR creation to assigned contributors#65849
rhshadrach wants to merge 2 commits into
pandas-dev:mainfrom
rhshadrach:pr_automation

Conversation

@rhshadrach

Copy link
Copy Markdown
Member

Claude generated; I have not reviewed most of this yet. Putting up so others can see the intended implementation via the docs changes here.

Comment on lines +49 to +57
await comment(`Thanks for your interest, @${user}! This issue is still labeled \`${blocking}\`, which means a maintainer needs to review it before work starts — so it isn't open to claim just yet. Please hold off on a PR until that label is removed. See the [contributing guide](${docsUrl}) for more.`);
return;
}
if (assignees.includes(user)) {
await comment(`You already have this one, @${user} — it's assigned to you and you're all set to work on it.`);
return;
}
if (assignees.length > 0) {
await comment(`Thanks @${user}! This issue is currently assigned to @${assignees[0]}, who's already on it — so to avoid two people doing the same work, please pick a different available issue for now. If @${assignees[0]} becomes inactive, the issue is released automatically and you'll be able to \`/take\` it then. See the [contributing guide](${docsUrl}) for more.`);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My only consideration here is that if multiple users don't follow the procedures (e.g. multiple /take commands), we'll have just as many of these comments which might start detracting from other comments discussing the issue

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.

2 participants