team-compass icon indicating copy to clipboard operation
team-compass copied to clipboard

Adopt 'enforce-labels' GitHub action from JupyterLab team

Open yuvipanda opened this issue 1 year ago • 5 comments

Inspired by this comment (https://github.com/jupyterhub/kubespawner/pull/793#issuecomment-1749428749) from @consideRatio, where adding labels to PRs is very helpful in creating changelogs. The JupyterLab team has a GH workflow (https://github.com/jupyterlab/jupyterlab/blob/main/.github/workflows/enforce-label.yml) that ensures this happens on all PRs, and surfaces it more explicitly when it's not.

We could adopt this as a way to minimize toil during releases, where the person doing releases might have to go and relabel old PRs.

yuvipanda avatar Oct 05 '23 18:10 yuvipanda

I like having this as a reminder for the PR merger. The only thing I don't like is seeing that red X as a PR contributor for something entirely out of my control and irrelevant to my contribution (when contributing to repos I don't have labeling permission on, that is). Maybe most folks don't get the same "You've done something wrong" visceral reaction from the red X that I do?

minrk avatar Oct 06 '23 08:10 minrk

Agree with Min - it would be nice if, instead of failure, it added a "to-be-traiged" label, or left a "To the Reviewer, please add labels" comment. Which is something we could try to contribute to the action, under a feature flag or something.

sgibson91 avatar Oct 06 '23 09:10 sgibson91

yeah i don't like the big red either.

+1 on moving it to a comment, perhaps one that even gets autodeleted after? I think that's how other projects do it

yuvipanda avatar Oct 06 '23 16:10 yuvipanda

I like the affirmative triage label, since that's easiest to look for, since you can visit is:merged label:triage to quickly view all the PRs that need a label (is:merged -label:bug OR -label:enhancement... is a lot harder)

minrk avatar Oct 06 '23 18:10 minrk

I like the idea of using labels too, it avoids sending comment notifications to everyone.

manics avatar Oct 06 '23 19:10 manics