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.

Sign up for GitHub

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

Jump to bottom

cargo doc: pass --cap-lints allow when documenting dependencies #5749

Closed
QuietMisdreavus opened this issue Jul 19, 2018 · 2 comments · Fixed by #5765
Closed

cargo doc: pass --cap-lints allow when documenting dependencies #5749

QuietMisdreavus opened this issue Jul 19, 2018 · 2 comments · Fixed by #5765
Labels
T-cargo Team: Cargo

Comments

@QuietMisdreavus
Copy link
Member

When rust-lang/rust#52354 lands, rustdoc will be able to accept --cap-lints to suppress lints when building documentation. Cargo should take advantage of this to pass --cap-lints allow when documenting dependencies, like how cargo build does the same. This will help contain the damage of rust-lang/rust#51468 and errors like it.
@QuietMisdreavus
Copy link
Member Author

Update: rust-lang/rust#52354 has just landed.

@joshtriplett joshtriplett added the T-cargo Team: Cargo label Jul 20, 2018
@ehuss
Copy link
Contributor

ehuss commented Jul 21, 2018

I have a patch ready, just waiting for a new nightly to be published.

@ehuss ehuss mentioned this issue Jul 22, 2018
Merged
bors added a commit that referenced this issue Jul 22, 2018
@bors
Auto merge of #5765 - ehuss:rustdoc-cap-lints, r=alexcrichton
ffcf739 
Support --cap-lints in rustdoc.

Fixes #5749.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
T-cargo Team: Cargo
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Support --cap-lints in rustdoc. ehuss/cargo
3 participants
@ehuss @joshtriplett @QuietMisdreavus