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

task: Added workflow for calling update-version-action #4805

Merged
merged 2 commits into from
Sep 25, 2023

Conversation

chriswk
Copy link
Contributor

@chriswk chriswk commented Sep 21, 2023

What

This adds a workflow which automatically triggers on published releases to update the version number for our version checker. In addition it adds a workflow dispatch, in case the version number ended up wrong after an automatic run (for instance when patching an earlier released minor).

Observations

Currently the version checker which receives the update only validates that the new version is also valid semver, it does not however check/verify that we're not suddenly telling it an older version is the newest (improvement available in the version updater repo).

@vercel
Copy link

vercel bot commented Sep 21, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
unleash-monorepo-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 22, 2023 9:34am
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
unleash-docs ⬜️ Ignored (Inspect) Visit Preview Sep 22, 2023 9:34am

Copy link
Contributor

@gastonfournier gastonfournier left a comment

Choose a reason for hiding this comment

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

When is this triggering? If it is with new enterprise releases, then I think this is perfectly fine.

Not sure if we want to keep the -main postfix:

"version": "5.4.0-main",
the main reason for this was to express that the version is in continuous change because we didn't have a mechanism for bumping it on formal releases, and bumping it on every change was time-consuming and could also create an infinite cycle with enterprise that continuously updates OSS on every change


on:
release:
types: [published]
Copy link
Contributor

Choose a reason for hiding this comment

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

Because of this I believe this will work just fine 👍

@chriswk chriswk merged commit fd9aeec into main Sep 25, 2023
15 checks passed
@chriswk chriswk deleted the task/addVersionCheckerUpdateAction branch September 25, 2023 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

2 participants