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

SRE-208 - Updating workflow to output NEXT_RELEASE_VERSION. #283

Merged
merged 2 commits into from
Oct 18, 2024

Conversation

jsclifford
Copy link
Contributor

@jsclifford jsclifford commented Oct 14, 2024

  • Added outputs NEXT_RELEASE_VERSION and NEXT_RELEASE_VERSION_NO_PREFIX.
  • Updating to latest git-version-lite action

@jsclifford jsclifford requested a review from a team as a code owner October 14, 2024 20:31
Copy link
Contributor

@mike-schenk mike-schenk left a comment

Choose a reason for hiding this comment

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

I don't think we should be committing package.json version numbers to the repo since we determine version numbers during our builds. My reasons are basically the same as what's documented by semantic-release here.

start-project is an example where the package version is not committed to source control: https://github.com/im-practices/start-project/blob/main/src/start-project/package.json

If there's another reason we need these outputs, I'll listen.

But regardless, I don't think it's worth running git-version-lite twice every time this reusable workflow is used when a very small subset of usages need the extra output.

As an alternative, we could expand git-version-lite itself to output the additional values directly.

@jsclifford
Copy link
Contributor Author

Ok I will update the action to output both versions in git-version lite.

@jsclifford
Copy link
Contributor Author

I updated git-version-lite to output the bare versions. im-open/git-version-lite#49 please review. Once merged I will update this workflow.

@jsclifford
Copy link
Contributor Author

@mike-schenk please review pull request. I have updated code per your request in your comments

@jsclifford jsclifford changed the title SRE-208 - Updating workflow to output nextversion without pre-release to update package.json in npm builds. SRE-208 - Updating workflow to output NEXT_RELEASE_VERSION. Oct 18, 2024
@jsclifford jsclifford merged commit 18ea518 into main Oct 18, 2024
@jsclifford jsclifford deleted the SRE-208-Adding-More-Outputs branch October 18, 2024 19:04
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.

2 participants