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

needless_doctest_main: handle correctly parse errors #6458

Merged
merged 3 commits into from
Dec 18, 2020

Conversation

ebroto
Copy link
Member

@ebroto ebroto commented Dec 15, 2020

Before this change, finding an error when parsing a doctest would make Clippy exit without emitting an error. Now we properly catch a fatal error and ignore it.

Also, if a doctest specifies an edition in the info line, it will be used when parsing it.

changelog: needless_doctest_main: handle correctly parse errors

Fixes #6022

@rust-highfive
Copy link

r? @Manishearth

(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 Dec 15, 2020
@Manishearth
Copy link
Member

@bors r+

thanks!

@bors
Copy link
Contributor

bors commented Dec 18, 2020

📌 Commit bb68ec6 has been approved by Manishearth

@bors
Copy link
Contributor

bors commented Dec 18, 2020

⌛ Testing commit bb68ec6 with merge 88323e8...

@bors
Copy link
Contributor

bors commented Dec 18, 2020

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: Manishearth
Pushing 88323e8 to master...

@bors bors merged commit 88323e8 into rust-lang:master Dec 18, 2020
@ebroto ebroto deleted the 6022_parse_doctest branch December 18, 2020 08:32
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.

clippy fails to check project but no error shown, due to doc comment
4 participants