-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Configuring Ranger #3931
Comments
hi @mfix22, thanks for reaching out. We did try to use it a while back on our PRs but were unable to get it to work. I still think we'd really benefit from the functionality, but we didn't really have time to dig into why it wasn't working/what we may have been missing/etc. If you have any insight on how we could get it working that would be great! One example of a PR where we tried: #3526 |
Hey @calebcartwright sorry to hear that! I would love to help make sure it is working correctly for you. I see that all the checks passed on your PR, so that is looking good ✅ Did you have any other protections on the On a fresh clone of the repo, I see that Ranger is working fine, mfix22#1 (comment), so I am assuming it has something to do with your other integrations, most likely CI/CD. If you could try again with a small/dummy PR, that would be great, otherwise I will try and set up CircleCI on my fork to better recreate the situation. |
We use quite a lot of branch protection rules on this repo. My assumption is that one of these rules is preventing repo ranger from being able to merge for us. We have the following rules enabled:
Any thoughts? While its nice that you've proactively contacted us about this, one of the reasons we've not followed this up at all is that its not very obvious how to contact you with a support query. Even if your code is not public, have you considered setting up a public issues-only repo or something? |
Just wanted to say @mfix22 that a merge queue we could kick off after an approving review that would take care of updating the branch, waiting for tests to pass (except LGTM and Docker, which take too long), and then squash-merging, would be super helpful for this project. I'd estimate it would save 1–2 hours of maintainer time a week. |
@chris48s this is great feedback, thank you. We have a public feedback form you can contact me at here: https://reporanger.com/feedback, however, I will also set up a public issues-only repo for tracking bugs/issues here: https://github.com/reporanger/feedback. |
@paulmelnikow that is great to hear — I would absolutely love to help you save those hours. If you add the "squash when passing" label to a PR, Ranger will automatically squash-merge the PR once all the required statuses are passing, following all integration status checks, branch protection rules, etc. I you can add the label to a branch in need of merging, I can investigate why it might not be being merged. |
Can you look at the example @calebcartwright posted? #3526 I just added it to #3880 which is already up to date, with tests already passing. I don't see an out-of-date branch ready to merge, though if someone sees one, let's try adding the label. |
On it 👌 |
#3880 has the necessary approvals. This is what we have under "restrict who can push to matching branches". Do we need to add a user for Ranger? |
Hey @paulmelnikow I think you are right! The public status listing (https://github.com/gitapi/repos/badges/shields/pulls/3880) shows that the PR has a Can you try adding Ranger to an approved list you mentioned above? We have never seen this specific set of branch blocking rules before so I really appreciate you are letting me debug this 🙂 |
I can only add GitHub users. Which user is it? |
@repo-ranger is the username. |
Looks like the PR was merged! |
Great! 👍 Thanks for working through this with me. I added another set of logging to our backend that will help me understand this issue faster in the future 👌 Now that we got it working, are there any other features you would like help automating? On other repos I maintain, I configure things like: automatically deleting HEAD branches, deleting ":+1:" comments, stuff like that 🙂 |
Awesome! Will it update an out-of-date branch before attempting to merge it? That's the thing we need for this to be useful. |
And would we need that to be done via a dependabot command on dependabot opened PRs to avoid merge conflicts, and if so can ranger support that? Something like: If author is dependabot AND branch out-of-date then comment |
We are actively building out that feature, but trying to figure out how to best fit it in with the rest of Ranger's config. I will ping in this thread, or in a new issue immediately once the feature ships. |
Thanks for helping with this. Even without the ability to auto- |
@chris48s done! reporanger.com/docs#e526831839a841748cc551d214f0c546 Also I have just pushed the feature that will keep a branch up to date as long as your branch protection rules require that. Would you like me to enable that feature for you repo? |
Nice. Yes. Lets give it a try. Thanks |
@mfix22 I noticed the merge queue is waiting on LGTM to run, which is not a required check, mostly because it is sometimes very slow! Can I request a feature that only the required checks must pass in order to merge? Or alternatively, that we can list certain checks to ignore? |
Hey @paulmelnikow 👋 looks like Ranger no longer has permission to merge your branches. Did a branch protection rule recently get updated? You might have to give Ranger write access to re-enable PR merging. You can find how to handle this under the "Branch Protection" section of the Repo Ranger docs |
Seems to have just merged one in #5026 no? |
Oh, okay. Great! |
Thanks for reaching out! We had an issue where maintainers couldn't click the "Update Branch" button. It might have been due to a setting on the PR or maybe it was some GitHub fluke. See #5008 (comment) Maybe that was related? |
That could be! Thanks @paulmelnikow. I'm just glad it is still working 👍 |
📋 Description
I noticed that you installed Ranger but have not yet used it to automate merging PRs, closing issues, or handling other tasks.
Have you had any trouble with it, or need any help configuring it? I am here to help! 👍
If not, feel free to close this issue (or have Ranger do it for you 🙂)
The text was updated successfully, but these errors were encountered: