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

Test for commit concurrency errors #6

Open
carterm opened this issue Dec 10, 2021 · 0 comments
Open

Test for commit concurrency errors #6

carterm opened this issue Dec 10, 2021 · 0 comments

Comments

@carterm
Copy link
Contributor

carterm commented Dec 10, 2021

Migrated from...
cagov/wordpress-to-github#74

Had an auto update run while a human update happened. Got a commit patch error...
Error: 422 error making request patch https://api.github.com/repos/cagov/drought.ca.gov/git/refs/heads/main: "Unprocessable Entity"
Suspect it has something to do with moving the head, after someone else moved the head. Need to account for that.

With the use of github-tree-push, this is now only happening in commit-only mode. Would happen if a commit completed, but then during the compare before the commit could be moved, something changed on the branch. Could be testing by putting a breakpoint after the commit happens and then changing something in the branch somewhere else.

This is the API it uses.
https://docs.github.com/en/rest/reference/git#update-a-reference

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

1 participant