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: Run update-versions during prepublishOnly npm step #668

Merged

Conversation

thewtex
Copy link
Member

@thewtex thewtex commented Oct 11, 2022

Get the version from package.json, which should be set by semantic-release.

@thewtex thewtex requested a review from floryst October 11, 2022 02:05
@thewtex thewtex mentioned this pull request Oct 11, 2022
Get the version from package.json, which should be set by
semantic-release.
@floryst
Copy link
Collaborator

floryst commented Oct 12, 2022

@thewtex Were you able to verify this will work with a dry run?

@thewtex
Copy link
Member Author

thewtex commented Oct 12, 2022

@floryst Yes, locally with npm publish --dry-run. I am not sure how to test with semantic-release.

@floryst
Copy link
Collaborator

floryst commented Oct 12, 2022

I would presume that semantic-release will have written out the correct version by the prepublish step.

@thewtex thewtex merged commit 093187e into InsightSoftwareConsortium:master Oct 12, 2022
@thewtex thewtex deleted the prepublish-update-versions branch October 12, 2022 21:36
@github-actions
Copy link

🎉 This PR is included in version 1.0.0-b.40 🎉

The release is available on:

Your semantic-release bot 📦🚀

thewtex added a commit to thewtex/itk-wasm that referenced this pull request Oct 21, 2022
The dependency on itk-wasm was `itk-wasm@0.0.0-semantically-released`
following InsightSoftwareConsortium#668
github-actions bot pushed a commit that referenced this pull request Oct 21, 2022
# [1.0.0-b.44](itk-wasm-v1.0.0-b.43...itk-wasm-v1.0.0-b.44) (2022-10-21)

### Bug Fixes

* **io-packages:** Update with main package semantic-release version ([6e62ba7](6e62ba7)), closes [#668](#668)
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.

None yet

3 participants