Skip to content

Mention the libc version changes in RELEASE_PROCEDURE.md #419

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

Merged
merged 1 commit into from
Sep 8, 2016
Merged

Mention the libc version changes in RELEASE_PROCEDURE.md #419

merged 1 commit into from
Sep 8, 2016

Conversation

fiveop
Copy link
Contributor

@fiveop fiveop commented Sep 8, 2016

r? @nix-rust/nix-maintainers

@posborne
Copy link
Member

posborne commented Sep 8, 2016

@homu r+

@homu
Copy link
Contributor

homu commented Sep 8, 2016

📌 Commit e9ede5d has been approved by posborne

@homu
Copy link
Contributor

homu commented Sep 8, 2016

⌛ Testing commit e9ede5d with merge af2e638...

homu added a commit that referenced this pull request Sep 8, 2016
Mention the libc version changes in RELEASE_PROCEDURE.md

r? @nix-rust/nix-maintainers
- Make a commit with a message like "Release v0.8.3" with the following
changes:
- In `CHANGELOG.md`, rename the Unreleased section to the new version
followed by the date of the release.
- In `Cargo.toml`, update the version to the new version.
- In `Cargo.toml`, change the libc dependency to the latest version.
Copy link
Member

Choose a reason for hiding this comment

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

add a mention that before committing we should make sure it builds and tests. well I guess this would be picked up by travis when the PR is made.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I had the same idea. It's on my todo list!

@kamalmarhubi
Copy link
Member

thanks! I had indeed left this out

@homu
Copy link
Contributor

homu commented Sep 8, 2016

☀️ Test successful - status

@homu homu merged commit e9ede5d into nix-rust:master Sep 8, 2016
@fiveop fiveop deleted the libc_version_release branch October 15, 2016 17:48
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.

4 participants