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

ci(triggers): change release process from tag push to release publish #3227

Merged
merged 3 commits into from
Mar 8, 2024

Conversation

ramin
Copy link
Contributor

@ramin ramin commented Mar 1, 2024

Updates the CI workflow trigger to run specifically from a release publish, vs pushing a tag. This will separate any testing we do with with any pre-release or temporary tags from triggering the GitHub workflow to generate a release. Instead, only the release published event should trigger this.

Wondertan
Wondertan previously approved these changes Mar 1, 2024
@ramin ramin added the kind:ci CI related PRs label Mar 1, 2024
walldiss
walldiss previously approved these changes Mar 4, 2024
MSevey
MSevey previously requested changes Mar 5, 2024
Copy link
Member

@MSevey MSevey left a comment

Choose a reason for hiding this comment

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

minor clean up of the if condition.

.github/workflows/ci_release.yml Outdated Show resolved Hide resolved
@ramin ramin dismissed stale reviews from walldiss and Wondertan via 826e715 March 5, 2024 17:04
Co-authored-by: Matthew Sevey <mjsevey@gmail.com>
@ramin ramin enabled auto-merge (squash) March 6, 2024 16:21
@ramin ramin merged commit e902680 into main Mar 8, 2024
27 checks passed
@ramin ramin deleted the ci/release-trigger branch March 8, 2024 10:09
renaynay pushed a commit to renaynay/celestia-node that referenced this pull request Apr 3, 2024
…celestiaorg#3227)

<!--
Thank you for submitting a pull request!

Please make sure you have reviewed our contributors guide before
submitting your
first PR.

Please ensure you've addressed or included references to any related
issues.

Tips:
- Use keywords like "closes" or "fixes" followed by an issue number to
automatically close related issues when the PR is merged (e.g., "closes
#123" or "fixes #123").
- Describe the changes made in the PR.
- Ensure the PR has one of the required tags (kind:fix, kind:misc,
kind:break!, kind:refactor, kind:feat, kind:deps, kind:docs, kind:ci,
kind:chore, kind:testing)

-->

Updates the CI workflow trigger to run specifically from a release
publish, vs pushing a tag. This will separate any testing we do with
with any pre-release or temporary tags from triggering the GitHub
workflow to generate a release. Instead, only the release published
event should trigger this.

---------

Co-authored-by: Matthew Sevey <mjsevey@gmail.com>
renaynay pushed a commit to renaynay/celestia-node that referenced this pull request Apr 3, 2024
…celestiaorg#3227)

<!--
Thank you for submitting a pull request!

Please make sure you have reviewed our contributors guide before
submitting your
first PR.

Please ensure you've addressed or included references to any related
issues.

Tips:
- Use keywords like "closes" or "fixes" followed by an issue number to
automatically close related issues when the PR is merged (e.g., "closes
#123" or "fixes #123").
- Describe the changes made in the PR.
- Ensure the PR has one of the required tags (kind:fix, kind:misc,
kind:break!, kind:refactor, kind:feat, kind:deps, kind:docs, kind:ci,
kind:chore, kind:testing)

-->

Updates the CI workflow trigger to run specifically from a release
publish, vs pushing a tag. This will separate any testing we do with
with any pre-release or temporary tags from triggering the GitHub
workflow to generate a release. Instead, only the release published
event should trigger this.

---------

Co-authored-by: Matthew Sevey <mjsevey@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:ci CI related PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants