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: prevent release.yml from running twice [skip ci] #831

Merged
merged 1 commit into from
Dec 13, 2024

Conversation

vinmassaro
Copy link
Contributor

Description of work

release.yml currently runs when a push is made to master, but should be changed to when a PR is merged, since semantic-release bumps the yalesites_profile version during the release with an additional commit, which is making the action run twice.

release.yml currently runs when a push is made to master, but should be changed to when a PR is merged, since semantic-release bumps the yalesites_profile version during the release with an additional commit, which is making the action run twice.
@vinmassaro vinmassaro requested a review from a team as a code owner December 13, 2024 17:52
Copy link
Contributor

@dblanken-yale dblanken-yale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Like it! That if statement looks much more along what we wanted to me.

Copy link

Visit Site

Created multidev environment pr-831 for yalesites-platform.

@vinmassaro vinmassaro merged commit 08d9938 into develop Dec 13, 2024
3 checks passed
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