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

Make doc_unsafe warn on unsafe traits as well #7734

Merged
merged 2 commits into from
Sep 28, 2021

Conversation

Manishearth
Copy link
Member

Fixes #7732

changelog: Make [doc_unsafe] warn on unsafe traits as well

@rust-highfive
Copy link

r? @camsteffen

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Sep 28, 2021
clippy_lints/src/doc.rs Outdated Show resolved Hide resolved
Co-authored-by: Cameron Steffen <cam.steffen94@gmail.com>
@camsteffen
Copy link
Contributor

@bors r+

@bors
Copy link
Contributor

bors commented Sep 28, 2021

📌 Commit 53c534d has been approved by camsteffen

@bors
Copy link
Contributor

bors commented Sep 28, 2021

⌛ Testing commit 53c534d with merge fb17671...

@bors
Copy link
Contributor

bors commented Sep 28, 2021

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: camsteffen
Pushing fb17671 to master...

@bors bors merged commit fb17671 into rust-lang:master Sep 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

missing_safety_doc should catch unsafe impl
4 participants