Skip to content

Rollup of 5 pull requests - #82182

Closed
GuillaumeGomez wants to merge 11 commits into
rust-lang:masterfrom
GuillaumeGomez:rollup-9ucodwa
Closed

Rollup of 5 pull requests#82182
GuillaumeGomez wants to merge 11 commits into
rust-lang:masterfrom
GuillaumeGomez:rollup-9ucodwa

Conversation

@GuillaumeGomez

Copy link
Copy Markdown
Member

Successful merges:

Failed merges:

r? @ghost
@rustbot modify labels: rollup

Create a similar rollup

GuillaumeGomezand others added 11 commits February 15, 2021 10:11
If we already have a slice, there is no need to get another full-range slice from that, just use the original.
clippy::redundant_slicing
ES5 checks
rustdoc is supposed to run on IE11 but someone reported me that it wasn't. I just confirmed it by using `es-check` with the `es5` option like this:
```
$ es-check es5 src/librustdoc/html/static/*.js
```
The PR fixes those issues and add CI checks to prevent regressions.
`@Mark-Simulacrum:` I added checks in the CI, but not sure if it's the correct way to do it. Any help on that side would be very appreciated!
r? `@Nemo157`
Fix typo in rustc_infer::infer::UndoLog
Also use double quotes.
…545, r=GuillaumeGomez
Add long explanation for E0545
Helps with rust-lang#61137
avoid full-slicing slices
If we already have a slice, there is no need to get another full-range slice from that, just use the original.
clippy::redundant_slicing
validation: fix invalid-fn-ptr error message
rust-lang#82061 changed the code here to print an `ImmTy` instead of a `ScalarMaybeUninit`; that was an accident. So go back to printing a `ScalarMaybeUninit`.
r? `@oli-obk`
@rustbotrustbot added the rollup A PR which is a rollup label Feb 16, 2021
@GuillaumeGomez

Copy link
Copy Markdown
MemberAuthor

@bors: r+ p=5

@bors

bors commented Feb 16, 2021

Copy link
Copy Markdown
Collaborator

📌 Commit 17f5463 has been approved by GuillaumeGomez

@borsbors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Feb 16, 2021
@Dylan-DPC-zz

Copy link
Copy Markdown

created a bigger rollup instead since the queue is longer now :D

@GuillaumeGomez
GuillaumeGomez deleted the rollup-9ucodwa branch February 16, 2021 15:00
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rollupA PR which is a rollupS-waiting-on-borsStatus: 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.

8 participants

@GuillaumeGomez@bors@Dylan-DPC-zz@rustbot@pierwill@jesusprubio@matthiaskrgr@RalfJung