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

Run release workflow after CI workflow #40

Merged
merged 1 commit into from
Jan 17, 2024
Merged

Run release workflow after CI workflow #40

merged 1 commit into from
Jan 17, 2024

Conversation

alemorcuq
Copy link
Contributor

With the current configuration, the release workflow and the CI workflow run in parallel. We would want the CI workflow to finish before the release workflow runs. With this PR:

  • The release workflow will run when a CI workflow completes in main
  • The release job in the workflow will only run if:
    • The CI workflow was successful, and
    • The head commit message for that workflow contains "Prepare release v"
  • A version check exits the job if version in plugin.yaml has not changed from the previous release (the latest tag equals the version in the file)

With the current configuration, the release workflow and the CI
workflow run in parallel. We would want the CI workflow to finish
before the release workflow runs. With this PR:

- The release workflow will run when a CI workflow completes in `main`
- The release job in the workflow will only run if:
  - The CI workflow was successful, and
  - The head commit message for that workflow contains "Prepare release v"
- A version check exits the job if version in `plugin.yaml` has not changed
from the previous release (the latest tag equals the version in the file)
Copy link
Contributor

@mpermar mpermar left a comment

Choose a reason for hiding this comment

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

Fantastic stuff!

@alemorcuq alemorcuq merged commit 2e91c94 into main Jan 17, 2024
2 checks passed
@alemorcuq alemorcuq deleted the release-after-ci branch January 17, 2024 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants