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 4 pull requests #59584

Merged
merged 13 commits into from
Mar 31, 2019
Merged

Rollup of 4 pull requests #59584

merged 13 commits into from
Mar 31, 2019

Conversation

Centril
Copy link
Contributor

@Centril Centril commented Mar 31, 2019

Successful merges:

Failed merges:

r? @ghost

stepnivlk and others added 13 commits March 16, 2019 19:16
Remove unnecessary comment
This commit extends the existing generic re-ordering diagnostic to
include any bounds on the generic parameter, thus producing correct
suggestions.
This commit updates the generic parameter re-ordering diagnostic to only
mention const generics if the feature is enabled.
…ths-libstd, r=oli-obk

libstd: deny(elided_lifetimes_in_paths)

r? @oli-obk
…i-obk

Mention `no merge policy` in the CONTRIBUTING guide

Issue: rust-lang#59233
Include bounds in generic re-ordering diagnostic

Fixes rust-lang#59508.

r? @estebank
cc @varkor
…ing, r=Centril

Distinguish message for external macros depending on error level

fixes rust-lang#57716

(I picked you because assigned to this issue.)
r? @estebank
@Centril
Copy link
Contributor Author

Centril commented Mar 31, 2019

@bors r+ p=4

@bors
Copy link
Contributor

bors commented Mar 31, 2019

📌 Commit fb8396d has been approved by Centril

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Mar 31, 2019
@bors
Copy link
Contributor

bors commented Mar 31, 2019

⌛ Testing commit fb8396d with merge a89c03a...

bors added a commit that referenced this pull request Mar 31, 2019
Rollup of 4 pull requests

Successful merges:

 - #58828 (libstd: deny(elided_lifetimes_in_paths))
 - #59234 (Mention `no merge policy` in the CONTRIBUTING guide)
 - #59572 (Include bounds in generic re-ordering diagnostic)
 - #59574 (Distinguish message for external macros depending on error level)

Failed merges:

r? @ghost
@bors
Copy link
Contributor

bors commented Mar 31, 2019

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

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Mar 31, 2019
@bors bors merged commit fb8396d into rust-lang:master Mar 31, 2019
@Centril Centril deleted the rollup branch March 31, 2019 17:19
@Centril Centril added the rollup A PR which is a rollup label Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. rollup A PR which is a rollup 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.

4 participants