semantic-release by abernier · Pull Request #194 · pmndrs/react-postprocessing

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@abernier

Copy link Copy Markdown

Member

@abernier abernier commented

May 1, 2023

"kill all humans"
— Bender

@v1rtl

Copy link Copy Markdown

Member

v1rtl commented

May 1, 2023

why reset the version?

@abernier abernier force-pushed the abernier/semantic-release branch from ba431f9 to fab7474 Compare

May 1, 2023 10:16

@abernier

Copy link Copy Markdown

Member Author

abernier commented

May 1, 2023

edited

Loading

why reset the version?

Semantic release takes care of updating the version just before publishing to NPM. There is no need to have it versioned in source code. Thanks to semantic-release, the version will already appear as the latest release:
image

This is also how it's done in drei and explained in more details here: https://semantic-release.gitbook.io/semantic-release/support/faq#why-is-the-package.jsons-version-not-updated-in-my-repository

v1rtl reacted with thumbs up emoji

@abernier

Copy link Copy Markdown

Member Author

abernier commented

May 1, 2023

edited

Loading

I think I'm done.

We could safely merge it into master: it won't trigger a release since only chore: commit messages in this PR (only feat: or fix: will).

Once merged, we could check the new Build and Release to npm pipeline just succeeds (without publishing any new version)

@drcmda drcmda merged commit 3a9998a into master

May 1, 2023

@drcmda

Copy link Copy Markdown

Member

drcmda commented

May 1, 2023

thanks @abernier !

@v1rtl v1rtl deleted the abernier/semantic-release branch

May 1, 2023 11:09

@github-actions

Copy link Copy Markdown

github-actions bot commented

May 1, 2023

🎉 This PR is included in version 2.9.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@abernier @v1rtl @drcmda