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/release.md: Add release documentation #1712

Merged
merged 7 commits into from
Aug 27, 2020
Merged

Conversation

mxinden
Copy link
Member

@mxinden mxinden commented Aug 18, 2020

In order to stay consistent across releases and also to enable possibly anyone with the correct access rights to release libp2p, this pull request adds a release documentation to the repository.

I have never cut a release of rust-libp2p. This documentation is based on past discussions and the git commit history. To reviewers: Please ensure this is in line with how you handled releasing for the last couple of versions.

@mxinden mxinden requested a review from romanb August 18, 2020 11:09
docs/release.md Outdated Show resolved Hide resolved
docs/release.md Outdated Show resolved Hide resolved
docs/release.md Outdated Show resolved Hide resolved
docs/release.md Outdated Show resolved Hide resolved
docs/release.md Outdated Show resolved Hide resolved
docs/release.md Outdated Show resolved Hide resolved
docs/release.md Outdated

1. Set the release date for each crate to be released in the respective
`CHANGELOG.md` by replacing `# X.Y.Z [unreleased]` with
`# X.Y.Z [yyyy-mm-dd]` and create a pull request against the rust-libp2p
Copy link
Contributor

Choose a reason for hiding this comment

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

I wonder why we insist on putting those dates in there? The version corresponds to a tag which maps to a commit which has a date. How often do people look at the date in a changelog?

Copy link
Member Author

Choose a reason for hiding this comment

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

I am fine either way. I just want to be consistent across all github.com/libp2p/rust-libp2p crates. @romanb thoughts?

Copy link
Contributor

Choose a reason for hiding this comment

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

I don't care about the dates in the changelogs much. That tradition was started here.

@mxinden
Copy link
Member Author

mxinden commented Aug 24, 2020

@twittner 17eda95 removes the requirement to include release dates in the changelog files.

Any other comments / suggestions?

@romanb romanb merged commit 293f2f8 into libp2p:master Aug 27, 2020
@romanb
Copy link
Contributor

romanb commented Aug 27, 2020

👍 Let's start with this. Thanks for the writeup.

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.

3 participants