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

Failed to update branch #100

Closed
Timer opened this issue Jul 25, 2019 · 8 comments
Closed

Failed to update branch #100

Timer opened this issue Jul 25, 2019 · 8 comments

Comments

@Timer
Copy link

Timer commented Jul 25, 2019

https://github.com/zeit/next.js/pull/8020/checks?check_run_id=177364070

There's no reason this branch should've failed to been updated AFAIK.

@chdsbd
Copy link
Owner

chdsbd commented Jul 25, 2019

These are the logs I have for it. Not sure why Github is saying that.

Jul 25 12:24:09 d.848cca00-415f-4d92-b718-b93e1bde6eaa app/web.1:  2019-07-25 16:24.09 could not update branch        repo=zeit/next.js#8020 res=<Response [404]> res_json={'message': 'Base does not exist', 'documentation_url': 'https://developer.github.com/v3/repos/merging/#perform-a-merge'} 
Jul 25 12:24:12 d.848cca00-415f-4d92-b718-b93e1bde6eaa app/web.1:  2019-07-25 16:24.12 could not update branch        repo=zeit/next.js#8020 res=<Response [404]> res_json={'message': 'Base does not exist', 'documentation_url': 'https://developer.github.com/v3/repos/merging/#perform-a-merge'} 
Jul 25 12:24:15 d.848cca00-415f-4d92-b718-b93e1bde6eaa app/web.1:  2019-07-25 16:24.15 could not update branch        repo=zeit/next.js#8020 res=<Response [404]> res_json={'message': 'Base does not exist', 'documentation_url': 'https://developer.github.com/v3/repos/merging/#perform-a-merge'} 
Jul 25 12:24:18 d.848cca00-415f-4d92-b718-b93e1bde6eaa app/web.1:  2019-07-25 16:24.18 could not update branch        repo=zeit/next.js#8020 res=<Response [404]> res_json={'message': 'Base does not exist', 'documentation_url': 'https://developer.github.com/v3/repos/merging/#perform-a-merge'} 
Jul 25 12:24:21 d.848cca00-415f-4d92-b718-b93e1bde6eaa app/web.1:  2019-07-25 16:24.21 could not update branch        repo=zeit/next.js#8020 res=<Response [404]> res_json={'message': 'Base does not exist', 'documentation_url': 'https://developer.github.com/v3/repos/merging/#perform-a-merge'} 

@chdsbd
Copy link
Owner

chdsbd commented Jul 25, 2019

It could be something to do with forks. I haven't played around with those before.

EDIT: Maybe kodiak is trying to update the fork, which it doesn't have permission to do.

@Timer
Copy link
Author

Timer commented Jul 25, 2019

Yup, just so you know these PRs have the "allow edits from maintainers" toggle enabled.

@chdsbd
Copy link
Owner

chdsbd commented Jul 25, 2019

Good to know. I'm going to close this in favor of #104. Has kodiak updated forks in the past for you?

@chdsbd chdsbd closed this as completed Jul 25, 2019
@Timer
Copy link
Author

Timer commented Jul 25, 2019

I don't believe so -- just trying to use Kodiak on an open source project instead of our internal apps.

@chdsbd
Copy link
Owner

chdsbd commented Jul 25, 2019

Thanks. That's useful information. I've documented this problem in the README#known-issues until #104 is addressed.

@chdsbd
Copy link
Owner

chdsbd commented Jul 27, 2019

@Timer It appears that it is only possible to update branches within the same repo through the Github API, so I think the best we can do for this issue is to display a warning to users for affected PRs (PRs from forked repos).

https://developer.github.com/v3/repos/merging/

@Timer
Copy link
Author

Timer commented Jul 28, 2019

Thanks for the update @chdsbd! Hopefully GitHub can fix this soon.

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

No branches or pull requests

2 participants