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(ci): update from published to released trigger for deployment epic #3351

Merged
merged 2 commits into from
May 1, 2024

Conversation

MSevey
Copy link
Member

@MSevey MSevey commented Apr 30, 2024

I realized that the published event is triggered by pre-releases too.
The released event is only triggered when we set the release to the latest release, which is what we want to track for deployments.

@github-actions github-actions bot added the external Issues created by non node team members label Apr 30, 2024
@MSevey MSevey added the kind:ci CI related PRs label Apr 30, 2024
@MSevey MSevey self-assigned this Apr 30, 2024
@ramin ramin enabled auto-merge (squash) May 1, 2024 07:37
@ramin ramin merged commit 40ee3c5 into main May 1, 2024
28 checks passed
@ramin ramin deleted the MSevey-patch-2 branch May 1, 2024 07:44
walldiss pushed a commit to walldiss/celestia-node that referenced this pull request May 7, 2024
celestiaorg#3351)

<!--
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
celestiaorg#123" or "fixes celestiaorg#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)

-->
I realized that the `published` event is triggered by pre-releases too.
The `released` event is only triggered when we set the release to the
latest release, which is what we want to track for deployments.

(cherry picked from commit 40ee3c5)
walldiss pushed a commit to walldiss/celestia-node that referenced this pull request May 7, 2024
celestiaorg#3351)

<!--
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
celestiaorg#123" or "fixes celestiaorg#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)

-->
I realized that the `published` event is triggered by pre-releases too.
The `released` event is only triggered when we set the release to the
latest release, which is what we want to track for deployments.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
external Issues created by non node team members kind:ci CI related PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants