Skip to content
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

Try to add 'completions' label to PRs automatically #10515

Closed
wants to merge 1 commit into from

Conversation

mqudsi
Copy link
Contributor

@mqudsi mqudsi commented May 22, 2024

The desired behavior is for a PR that modifies share/completions/.fish and does not modify any other paths except ignoring changes to share/functions/.fish gets a 'completions' label.

(There's another approach I was looking at first that can automatically set the 'fish next-3.x' milestone as well by doing all this manually via the api instead of using the labeler@v5 action, but it requires setting up GitHub secret tokens and I don't feel like doing that right now.)

I'm not sure how to test this 😅

@mqudsi mqudsi force-pushed the auto_label_prs branch 6 times, most recently from 456dd52 to 6a96a4e Compare May 22, 2024 14:34
@zanchey
Copy link
Member

zanchey commented May 22, 2024

One approach to testing is to use your fork - does mean you need to turn issues/PRs on temporarily but I think it can even be done in a private fork

@mqudsi
Copy link
Contributor Author

mqudsi commented May 22, 2024

Trying that now. I have to merge this into my master branch because that name is hard-coded in the yaml file, though.

The desired behavior is for a PR that modifies share/completions/*.fish and does
not modify any other paths except ignoring changes to share/functions/*.fish
gets a 'completions' label.

(There's another approach I was looking at first that can automatically set the
'fish next-3.x' milestone as well by doing all this manually via the api instead
of using the labeler@v5 action, but it requires setting up GitHub secret tokens
and I don't feel like doing that right now.)
@mqudsi
Copy link
Contributor Author

mqudsi commented May 22, 2024

Got this working in 53eeb8a but I was nerd-sniped into figuring out how to set the milestone too so I'm opening up a new PR that does both.

@mqudsi mqudsi closed this May 22, 2024
@mqudsi mqudsi deleted the auto_label_prs branch July 11, 2024 17:06
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.

None yet

2 participants