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

Specify Rust minimum version requirements #522

Merged
merged 1 commit into from
Feb 24, 2017
Merged

Conversation

Susurrus
Copy link
Contributor

I think this is important for 2 reasons:

  1. Users know if they can use this library
  2. Developers know what Rust features they can use in merge requests

What this doesn't do is establish any kind of policy for raising the Rust requirements, which is something worth addressing in the near-future.

@kamalmarhubi
Copy link
Member

Thanks! There's an RFC issue to figure out the policy: nix-rust/rfcs#4

@homu r+

@homu
Copy link
Contributor

homu commented Feb 24, 2017

📌 Commit 56a56dd has been approved by kamalmarhubi

homu added a commit that referenced this pull request Feb 24, 2017
Specify Rust minimum version requirements

I think this is important for 2 reasons:

1) Users know if they can use this library
2) Developers know what Rust features they can use in merge requests

What this doesn't do is establish any kind of policy for raising the Rust requirements, which is something  worth addressing in the near-future.
@homu
Copy link
Contributor

homu commented Feb 24, 2017

⌛ Testing commit 56a56dd with merge c3905ee...

@homu
Copy link
Contributor

homu commented Feb 24, 2017

☀️ Test successful - status

@homu homu merged commit 56a56dd into nix-rust:master Feb 24, 2017
@Susurrus Susurrus deleted the readme branch February 24, 2017 21:36
@fiveop
Copy link
Contributor

fiveop commented Feb 25, 2017

If bitflags 0.7 does not run on rust 1.7 (see all our 1.7 builds breaking in the CI). >= 1.7 is certainly wrong.

@Susurrus
Copy link
Contributor Author

@fiveop Indeed, but that's what I was told! Anyways, I've determined that the min version is actually 1.9 and submitted a PR (#525) to resolve that for the docs and CI.

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