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] Trigger publish job on tag creation, try again to publish #199

Merged
merged 2 commits into from
Feb 2, 2024

Conversation

marcdumais-work
Copy link
Contributor

@marcdumais-work marcdumais-work commented Feb 1, 2024

When we previously created the v0.2.0 release using the RELEASE file, the corresponding GitHub release and git tag were created. However the publish job was not triggered. Testing on my fork, I concluded that the workflow needs to trigger on "create" tag, not on "push". One commit in this PR fixes this.

A slight side-effect of triggering the ci-cd workflow on "create" events is that it will now run also on branch creation. So it may run a bit more often than before. I think this is preferable for now, vs having complex "if:" conditions on each job in the workflow.

Also, in a separate commit, we step the tag in file RELEASE to v0.2.1 to trigger a new release, which this time will hopefully trigger the publish job and successfully publish the extension to open-vsx. Afterwards, I plan to delete the (failed) v0.2.0 release, after copying its generated release notes to the new release

When we previously created the v0.2.0 release using the RELEASE file,
the corresponding GitHub release and git tag were created. However the
publish job was not triggered. Testing on my fork, I concluded that
the workflow needs to trigger on "create" tag, not on "push".

A slight side-effect of triggering the ci-cd workflow on "create" events
is that it will now run also on branch creation. So it may run a bit more
often than before. I think this is preferable for now, vs having complex
"if:" conditions on each job in the workflow.

This commit fixes this.

Signed-off-by: Marc Dumais <[email protected]>
This commit steps the tag in file RELEASE to v0.2.1 to trigger a new release,
which this time will hopefully trigger the publish job and successfully
publish the extension to open-vsx. Afterwards,  I plan to delete the (failed)
v0.2.0 release, and copy its generated release notes to the new release

Signed-off-by: Marc Dumais <[email protected]>
Copy link

github-actions bot commented Feb 1, 2024

RELEASE

The following 1 GitHub releases will be created once this pull request got merged.

Release v0.2.1 with changes since v0.2.0

  • Trigger a new release: v0.2.1 f974ded
  • [CI] Have publish CI job trigger on tag creation 0fc7e93

Copy link
Collaborator

@bhufmann bhufmann left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for the release.

@marcdumais-work marcdumais-work merged commit 8cf40dd into master Feb 2, 2024
5 of 6 checks passed
@marcdumais-work marcdumais-work deleted the fix-publish-ovsx branch February 2, 2024 15:56
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.

3 participants