Skip to content

Rollup of 6 pull requests - #59991

Merged
bors merged 15 commits into
rust-lang:masterfrom
Centril:rollup-bqxt4w3
Apr 15, 2019
Merged

Rollup of 6 pull requests#59991
bors merged 15 commits into
rust-lang:masterfrom
Centril:rollup-bqxt4w3

Conversation

@Centril

Copy link
Copy Markdown
Contributor

Successful merges:

Failed merges:

r? @ghost

agnxyand others added 15 commits April 6, 2019 18:04
This uplifts `get_def_path` from Clippy. This is a follow up on the
implementation of internal lints: rust-lang#59316
The internal lint implementation also copied the implementation of the
`AbsolutePathPrinter`. To get rid of this code duplication this also
uplifts the `get_def_path` function from Clippy.
This also renames `match_path` to `match_def_path`, as it was originally
named in Clippy.
Add must_use annotations to Result::is_ok and is_err
Discussed in rust-lang#59610
Add summary and reference to Rust trademark guide
Fixrust-lang#53287
cc @steveklabnik
…ishearth
Uplift `get_def_path` from Clippy
cc rust-lang/rust-clippy#3926
cc rust-lang#59738
This uplifts `get_def_path` from Clippy. This is a follow up on the
implementation of internal lints: rust-lang#59316
The internal lint implementation also copied the implementation of the
`AbsolutePathPrinter`. To get rid of this code duplication this also
uplifts the `get_def_path` function from Clippy.
This also renames `match_path` to `match_def_path`, as it was originally
named in Clippy.
r? @Manishearth
bump stdsimd; make intra_doc_link_resolution_failure an error again; make lints more consistent
I made `intra_doc_link_resolution_failure` warn so that it would properly respect `deny-warnings = false` in `config.toml`. `#[warn]` still become errors with `-D warnings` so I thought this was fine.
Turns out however that we don't pass `-D warnings` when running rustdoc, so for additional rustdoc-lints we need to set them to `deny`.
Also sue the opportunity to make the lint flags more consistent between libcore, liballoc, libstd.
Cc @gnzlbg for the *big* stdsimd update.
…dreavus
rustdoc: Remove default keyword from re-exported trait methods
Fixesrust-lang#59977
r? @QuietMisdreavus
As this fixes a stable to beta regression, could it be backported?
@Centril

Copy link
Copy Markdown
ContributorAuthor

@bors r+ p=6

@bors

bors commented Apr 15, 2019

Copy link
Copy Markdown
Collaborator

📌 Commit 6434fe9 has been approved by Centril

@borsbors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Apr 15, 2019
@bors

bors commented Apr 15, 2019

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 6434fe9 with merge 07133ac...

bors added a commit that referenced this pull request Apr 15, 2019
Rollup of 6 pull requests
Successful merges:
- #59648 (Add must_use annotations to Result::is_ok and is_err)
- #59748 (Add summary and reference to Rust trademark guide)
- #59779 (Uplift `get_def_path` from Clippy)
- #59955 (bump stdsimd; make intra_doc_link_resolution_failure an error again; make lints more consistent)
- #59978 (rustdoc: Remove default keyword from re-exported trait methods)
- #59989 (Fix links to Atomic* in RELEASES.md)
Failed merges:
r? @ghost
@bors

bors commented Apr 15, 2019

Copy link
Copy Markdown
Collaborator

☀️ Test successful - checks-travis, status-appveyor
Approved by: Centril
Pushing 07133ac to master...

@borsbors added the merged-by-bors This PR was explicitly merged by bors. label Apr 15, 2019
@bors
bors merged commit 6434fe9 into rust-lang:masterApr 15, 2019
@Centril
Centril deleted the rollup-bqxt4w3 branch April 15, 2019 19:26
@CentrilCentril added the rollup A PR which is a rollup label Oct 2, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged-by-borsThis PR was explicitly merged by bors.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

@Centril@bors@agnxy@alex@flip1995@RalfJung@ollie27@dima74