-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[service] Automatically Label Pull Requests with Megre Conflicts #4725
Conversation
Action label conflict
Don't mind all my test commits. |
Changes not allowed in build 1:
Split them in several pull requests if you are making changes in more than one folder. |
Hi, first of all, thanks! 🤟 This is not targeting exactly the specific problem we'd like to avoid in The problem arises when a PR can be merged (it has no conflicts), but it will create a new recipe revision when merged to Our problem is that the merge might create an invalid recipe (and also the time it takes to build) even if it mergeable. To avoid these situations, usually, you can require a rebase/update before merging, and also the CI systems provide a branch indexing feature (Jenkins terminology) to trigger again the PR merging to The implementation should check the repository files and see if, after merging to A Github action could do this job, adding labels like "This PR needs to be updated", our CI can check the label and ignore the PR... o trigger it again. This is different from the About this action, it is useful to add the label, but probably posting a comment in the PR that will trigger a notification to the user is also needed. wdyt? |
Ahh thank you for the detailed description, I think I better understood the problem before I had done this small project 😀 but it's more clear now. It's really eric's bot but comparing PRs to master.
Very first though I had. prince-chrismc/label-merge-conflicts-action#13 The labels are most helpful for bots. I'll need time to digest this but my first reaction is this might not obtainable/practical in a GitHub action. Actions are webhooks called after something has occurred. What CCI needs is before.
Since there's an unknown time that each PR is open, 1 day to 1 year, with only ~20 (0.2 to 0.25) active PRs open at any one time there would be a lot of waste calculation time given there's 5 to 10 merges a day. Not sure which organization plan you are on but that might be pricey. The technical challenge is whether running after each change the action can detect is sufficient to keep the PRs up to date. I've flipped from not to yes to maybe trying to write this. Would it be possible to have the bot, when considering to merge, calculate the This is a very interesting problem, I might come back with a second proposal now that the requirements are clear in my head. |
🎁 For CCI... this is probably the first step in making the comment below possible. Demo prince-chrismc#12
Originally posted by @jgsogo in #4577 (comment)
Optionally you may wish to use the @conan-center-bot PAT for the action so the changes will have that name.
Lastly there is https://docs.github.com/en/github/administering-a-repository/about-protected-branches#require-status-checks-before-merging but I am not sure how helpful it is with the CCI bot doing all the work.