Skip to content

Adjust for Stable features lint #28

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 2 commits into from
Nov 3, 2015

Conversation

Ryman
Copy link
Contributor

@Ryman Ryman commented Nov 2, 2015

Deny warnings doesn't seem very future-proofed, perhaps you want to deny(warnings) specifically on travis or something via cargo rustc -- -F warnings?

For reference, this killed the nickel build when cargo doc was being run.

Ryman added 2 commits November 2, 2015 20:58
This can affect downstream crates when a new nightly version is released
@Manishearth
Copy link
Owner

That ... shouldn't be happening. Cargo caps lints for dependencies to warn. Are you on an updated cargo?

@Ryman
Copy link
Contributor Author

Ryman commented Nov 2, 2015

@Manishearth It was unexpected to me too, I assume the attribute overrides it? Here's a log which is cargo 0.7.0-nightly (8acff56 2015-10-31).

I guess it might just be an oversight that cargo doc doesn't pass cap-lints?

@Manishearth
Copy link
Owner

No, it's not supposed to. That's the point of capping lints 😄

File a cargo bug, I guess.

@Ryman
Copy link
Contributor Author

Ryman commented Nov 3, 2015

Pushed a fix upstream, cargo didn't seem to be the correct place for it (would have required considerable change).

Do you want me to drop the second commit as it should in theory get fixed in future? Will still require the first commit to allow downstream to not fail though so it would be good if we could merge that!

Manishearth added a commit that referenced this pull request Nov 3, 2015
Adjust for Stable features lint
@Manishearth Manishearth merged commit 5a1ec93 into Manishearth:master Nov 3, 2015
bors added a commit to rust-lang/rust that referenced this pull request Nov 3, 2015
This sets the `cap-lints` setting to 'allow' for all doc compilations. There's precedent for this as rustdoc [already whitelists unstable code](https://github.com/rust-lang/rust/blob/master/src/librustdoc/core.rs#L112) when compiling documentation, with the expectation being that a regular compile will complain about any problems. I think the same justification applies here.

Problem case in the wild: Manishearth/compiletest-rs#28

r? @Manishearth
critiqjo pushed a commit to critiqjo/rustdoc that referenced this pull request Dec 16, 2016
This sets the `cap-lints` setting to 'allow' for all doc compilations. There's precedent for this as rustdoc [already whitelists unstable code](https://github.com/rust-lang/rust/blob/master/src/librustdoc/core.rs#L112) when compiling documentation, with the expectation being that a regular compile will complain about any problems. I think the same justification applies here.

Problem case in the wild: Manishearth/compiletest-rs#28

r? @Manishearth
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