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

Give a hard error if -Zrustdoc-scrape-examples is missing a flag #10043

Merged
merged 1 commit into from
Nov 20, 2021

Conversation

jyn514
Copy link
Member

@jyn514 jyn514 commented Nov 5, 2021

It's the same as if the option wasn't passed, and it makes it difficult to
figure out why rustdoc isn't generating examples.

Fixes #10042 (cc @willcrichton)

@rust-highfive
Copy link

r? @ehuss

(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 Nov 5, 2021
It's the same as if the flag wasn't passed, and it makes it difficult to
figure out why rustdoc isn't generating examples.
@willcrichton
Copy link
Contributor

Looks good to me, thanks :-)

@ehuss
Copy link
Contributor

ehuss commented Nov 20, 2021

Thanks, sorry for the slow review. I have notification overload.

@bors r+

@bors
Copy link
Collaborator

bors commented Nov 20, 2021

📌 Commit 8713cd7 has been approved by ehuss

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 20, 2021
@bors
Copy link
Collaborator

bors commented Nov 20, 2021

⌛ Testing commit 8713cd7 with merge 7a947ae...

@jyn514
Copy link
Member Author

jyn514 commented Nov 20, 2021

No worries! Thanks for the review :)

@bors
Copy link
Collaborator

bors commented Nov 20, 2021

☀️ Test successful - checks-actions
Approved by: ehuss
Pushing 7a947ae to master...

@bors bors merged commit 7a947ae into rust-lang:master Nov 20, 2021
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Nov 22, 2021
Update cargo

7 commits in ad50d0d266213e0cc4f6e526a39d96faae9a3842..e1fb17631eb1b3665cdbe45b1c186111577ef512
2021-11-17 18:36:37 +0000 to 2021-11-22 16:53:06 +0000
- re-enable lto_build test on 32-bit MSVC (rust-lang/cargo#10110)
- Fix a couple issues with cyclic features and dev-dependencies (rust-lang/cargo#10103)
- Add --message-format for install command (rust-lang/cargo#10107)
- Update curl dependency, remove M1 macOS build error note (rust-lang/cargo#10106)
- Make clippy happy (rust-lang/cargo#10105)
- Give a hard error if `-Zrustdoc-scrape-examples` is missing a flag (rust-lang/cargo#10043)
- silly fix, pointer to the empty slice (rust-lang/cargo#10097)
@ehuss ehuss added this to the 1.58.0 milestone Feb 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

-Zrustdoc-scrape-examples silently does nothing
5 participants