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

Bump mustache to 0.7.0 to fix build on nightly rustc #1

Merged
merged 1 commit into from
Jun 1, 2017
Merged

Bump mustache to 0.7.0 to fix build on nightly rustc #1

merged 1 commit into from
Jun 1, 2017

Conversation

petrochenkov
Copy link
Contributor

Hi.
rust-lang/rust#42136 is going to turn some old compatibility warnings in rustc into hard errors.
crates.io testing discovered that this crate uses an old version of mustache affected by one of those errors.
This PR updates mustache to 0.7.0, which is the minimal version building successfully with rust-lang/rust#42136

bors added a commit to rust-lang/rust that referenced this pull request Jun 1, 2017
Turn sufficiently old compatibility lints into hard errors

It's been almost 7 months since #36894 was merged, so it's time to take the next step.

[breaking-change], needs crater run.

PRs/issues submitted to affected crates:
https://github.com/alexcrichton/ctest/pull/17
Sean1708/rusty-cheddar#55
m-r-r/helianto#3
azdle/virgil#1
rust-locale/rust-locale#24
mneumann/acyclic-network-rs#1
reem/rust-typemap#38

cc https://internals.rust-lang.org/t/moving-forward-on-forward-compatibility-lints/4204
cc #34537 #36887
Closes #36886
Closes #36888
Closes #36890
Closes #36891
Closes #36892
r? @nikomatsakis
@azdle azdle merged commit a0ff875 into azdle:master Jun 1, 2017
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Jun 1, 2017
@petrochenkov
Copy link
Contributor Author

@azdle
Could you also publish the updated version to crates.io as virgil-0.2.2?

@azdle
Copy link
Owner

azdle commented Jun 3, 2017

Done, I just wanted to fix one other thing and update the rest of the deps before pushing the next version.

@petrochenkov
Copy link
Contributor Author

Thanks!

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