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

Fix: Creating deployments for another repository fails with Not Found #69

Merged

Conversation

liamoneill
Copy link
Contributor

This PR fixes the action from failing when creating a deployment for another repo then where the workflow is running from.

Error from logs:

Error: HttpError: Not Found
Error: Error creating GitHub deployment: Not Found

This was caused by the createDeploymentStatus request hardcoding the repo & owner from context instead of using the repo that deployment was created under.

I've tested the change in workflows that both create deployments on themselves and other repos and the fix applied to be working as expected.

@liamoneill liamoneill changed the title Use the same owner & repo for updating the deployment status Fix: Creating deployments for another repository fails with Error creating GitHub deployment: Not Found Aug 18, 2022
@liamoneill liamoneill changed the title Fix: Creating deployments for another repository fails with Error creating GitHub deployment: Not Found Fix: Creating deployments for another repository fails with Not Found Aug 18, 2022
@liamoneill
Copy link
Contributor Author

Hey @chrnorm, thanks for your work on this action! Are you still accepting fixes? Would love to get this upstream if possible.

@chrnorm
Copy link
Owner

chrnorm commented Sep 7, 2022

Hey @liamoneill - my apologies for the delay here. Thankyou so much for the fix!

@chrnorm chrnorm merged commit 81b5fe7 into chrnorm:main Sep 7, 2022
@liamoneill liamoneill deleted the fix/use-input-repo-for-updating-status branch September 7, 2022 23:03
@liamoneill
Copy link
Contributor Author

Hey @liamoneill - my apologies for the delay here. Thankyou so much for the fix!

No apology necessary! Thanks so much for the awesome work!

@chrnorm
Copy link
Owner

chrnorm commented Sep 7, 2022

@liamoneill this has been released in v2.0.4 😁

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

Successfully merging this pull request may close these issues.

2 participants