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: remove duplicate builds on merge queue and push #1131

Merged
merged 1 commit into from
Apr 14, 2024

Conversation

p5
Copy link
Member

@p5 p5 commented Apr 13, 2024

We are building and pushing to GHCR when a PR is added to a merge queue (merge_group), and also after the PR has been successfully merged into main (push).

To me, this seems like we are running a build when we do not need to, and unnecessarily taking up GHA runners.

This PR removes the builds after a commit has been merged to main and testing since these are ran during merge queue.

@p5 p5 requested a review from castrojo as a code owner April 13, 2024 23:22
@p5
Copy link
Member Author

p5 commented Apr 13, 2024

I've cancelled builds because they are not required for this PR.

@p5 p5 requested a review from m2Giles April 13, 2024 23:53
@p5
Copy link
Member Author

p5 commented Apr 14, 2024

Going to bypass branch protection here. No need for new builds.

@p5 p5 merged commit 0686642 into main Apr 14, 2024
2 of 70 checks passed
@p5 p5 deleted the remove-gha-trigger-push branch April 14, 2024 00:11
awesomekyle pushed a commit to awesomekyle/bluefin that referenced this pull request Apr 24, 2024
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