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

chore: mutation workflow and owner comment workflow broken on forks #5820

Merged
merged 1 commit into from
Feb 29, 2024

Conversation

MarkMcCulloh
Copy link
Contributor

@MarkMcCulloh MarkMcCulloh commented Feb 29, 2024

Fixes #5115

The mutation workflow is using the wrong head for forks when the repo name is wing (which it usually is). This worked before, I suspect the API changed, but in any case this fixes things.

Also updated the initial PR comment workflow to work on forks.

Tested in https://github.com/MarkMcCulloh/wing-distributed-workflow

By submitting this pull request, I confirm that my contribution is made under the terms of the Wing Cloud Contribution License.

@MarkMcCulloh MarkMcCulloh requested a review from a team as a code owner February 29, 2024 15:50
@staycoolcall911
Copy link
Contributor

Nice! - @MarkMcCulloh can you please add that it "Fixes #5115"?

Copy link
Contributor

mergify bot commented Feb 29, 2024

Thanks for contributing, @MarkMcCulloh! This PR will now be added to the merge queue, or immediately merged if mark/mutation-fix-fork is up-to-date with main and the queue is empty.

@mergify mergify bot merged commit 13b79ea into main Feb 29, 2024
16 checks passed
@mergify mergify bot deleted the mark/mutation-fix-fork branch February 29, 2024 18:19
@monadabot
Copy link
Contributor

Congrats! 🚀 This was released in Wing 0.59.24.

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.

PR creation comment CI job fails if a pull request is made from a fork
4 participants