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

Rollup of 6 pull requests #83658

Closed
wants to merge 16 commits into from

Conversation

Dylan-DPC-zz
Copy link

Successful merges:

Failed merges:

r? @ghost
@rustbot modify labels: rollup

Create a similar rollup

camelid and others added 16 commits March 26, 2021 19:50
And, now that we do that, we can remove the explanatory note since the
error span should make it clear what the disambiguator is.
escape_ascii take 2

The previous PR, rust-lang#73111 was closed for inactivity; since I've had trouble in the past reopening closed PRs, I'm just making a new one.

I'm still running the tests locally but figured I'd open the PR in the meantime. Will fix whatever errors show up so we don't have to wait again for this.

r? `@m-ou-se`
… r=jyn514

Lint on unknown intra-doc link disambiguators
…egg-test, r=lcnr

Add a regression test for issue-82792

Closes rust-lang#82792

r? `@lcnr`
…span, r=RalfJung

Remove a FIXME resolved by rust-lang#73578

r? `@RalfJung`
Wrap non-pre code blocks

Fix rust-lang#83550 regression

```
$ cargo new --lib whitespace && cd whitespace && echo '//! `"   foo   "`' > src/lib.rs && cargo doc --open
```

Before

![](https://user-images.githubusercontent.com/89623/112713498-c0dfc200-8ed5-11eb-8c57-efdf26372e74.png)

After

![](https://user-images.githubusercontent.com/89623/112713538-f08eca00-8ed5-11eb-8a98-675179f60ae2.png)

r? `@GuillaumeGomez`
cc `@mgeisler`
@rustbot rustbot added the rollup A PR which is a rollup label Mar 29, 2021
@Dylan-DPC-zz Dylan-DPC-zz deleted the rollup-ctv5my6 branch March 30, 2021 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants