-
Notifications
You must be signed in to change notification settings - Fork 21
Question concerning usage of this app #43
Comments
I have the same expectations as @gep13. It would be awesome if autorebase could meet those expectations. 😄 |
Hi, indeed I never tested the app on forks. I will probably have to make some developments on the upstream libraries github-cherry-pick and github-rebase to make it work. Not sure yet if it will be a quick fix or not but I'm determined to support the fork use case. |
@tibdex woo hoo! Let us know if you need any beta testers. Happy to help out! |
So, I investigated a bit and I have bad news. When making the necessary changes to support fork-PRs where the HEAD branch is from another repo (one that forked the original one), the Autorebase GitHub App doesn't have permission to force-push on this HEAD branch. Even if the author of the PR checked the "Allow edits from maintainers" box. The error is:
The rejected request is this one. Even if the owner of the fork were to install Autorebase on its repo, the GitHub App installation ID would be different so I'm quite sure it wouldn't help. So I guess supporting fork-PRs cannot be done with the current design. |
@tibdex thank you for taking the time to investigate this! To help me understand the infrastructure a little bit... What is the difference between me as a user force pushing to the branch on the fork, which is allowed, and the autorebase app? |
I don't know precisely how GitHub handles permissions for the "Allow edits from maintainers" option but I guess it only works for real users (actual accounts) and not GitHub applications. |
Yeah, that could well be it. |
If I provide you with an OAuth token to use I think that you will have permission to act on my behalf and rebase from a fork if "Allow edits from maintainers" is checked. So when we register you'll need to get a token from the user using the web flow authentication and you'll be able to use that token to impersonate the user. |
I was really excited when I found this app as I thought that it would be a perfect helper for the workflow that we employ on a number of repositories.
However, the first attempt at using it, didn't work as I would have expected:
cake-build/website#609 (comment)
However, I thought that the issue here was due to the fact that the PR was coming from the develop branch on the fork, rather than a branch on the fork. However, I have just tried again with a branch on a fork, and I got the same result:
cake-build/website#635 (comment)
Are my expectations of what this app can do wrong?
Normally, in this situation, the steps that I would follow manually would be:
And I was hoping that this app would be able to help with this. Is my expectations incorrect?
The text was updated successfully, but these errors were encountered: