Skip to content

Conversation

camsteffen
Copy link
Contributor

@camsteffen camsteffen commented May 26, 2021

changelog: Fix [missing_docs_in_private_items] false negative when the item has any #[name = "value"] attribute

Closes #7247 (decided not to use the rustc method since it calls Session::check_name, which is for rustc only)

@rust-highfive
Copy link

r? @flip1995

(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 May 26, 2021
@camsteffen camsteffen changed the title Fix missing_docs_in_private_items false positive Fix missing_docs_in_private_items false negative May 26, 2021
@flip1995
Copy link
Member

@bors r+

Thanks!

@bors
Copy link
Contributor

bors commented May 27, 2021

📌 Commit c21b965 has been approved by flip1995

@bors
Copy link
Contributor

bors commented May 27, 2021

⌛ Testing commit c21b965 with merge 2fa9362...

@bors
Copy link
Contributor

bors commented May 27, 2021

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: flip1995
Pushing 2fa9362 to master...

@bors bors merged commit 2fa9362 into rust-lang:master May 27, 2021
@camsteffen camsteffen deleted the has-doc-fp branch July 8, 2021 22:01
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.

Use rustc_lint::builtin::has_doc
4 participants