Skip to content

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
- 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