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

docs: update release steps #705

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

g-plane
Copy link
Contributor

@g-plane g-plane commented Feb 21, 2025

Original guide doesn't reflect the switching to release-plz, and this PR updates it.

Copy link

netlify bot commented Feb 21, 2025

Deploy Preview for salsa-rs canceled.

Name Link
🔨 Latest commit 9e7d106
🔍 Latest deploy log https://app.netlify.com/sites/salsa-rs/deploys/67b7dd5cb874c300086947b3

@MichaReiser
Copy link
Contributor

To me it seems that release-plz now immediately creates a PR with all changes since the last release and bumps the version number automatically. All that seems necessary is to merge that PR?

@g-plane
Copy link
Contributor Author

g-plane commented Feb 21, 2025

I think no. Should I remove this section in readme?

@MichaReiser
Copy link
Contributor

I'd find instruction useful on how to do a release because it's currently unclear to me what needs doing. I just noticed the automatically created release PR, that makes me wonder if manually bumping the version is still necessary

@g-plane
Copy link
Contributor Author

g-plane commented Feb 21, 2025

Maybe no need to do it manually, but I didn't try release-plz before.

Copy link
Contributor

@MichaReiser MichaReiser left a comment

Choose a reason for hiding this comment

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

Let's merge this and we can iterate on the content as we gain more experience with release-plz

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