Skip to content

Conversation

fakenine
Copy link
Contributor

@fakenine fakenine commented Jul 7, 2019

normalize use of backticks in compiler messages for librustc_typecheck

#60532

@rust-highfive
Copy link
Contributor

Some changes occurred in diagnostic error codes

cc @GuillaumeGomez

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 7, 2019
@Centril
Copy link
Contributor

Centril commented Jul 7, 2019

@fakenine Can you cherry-pick the two first commits into different PRs? The last commit should ideally also be broken into more based on files/feature gates this is changing to make it easier to land the PRs.

(Also, please avoid pinging people in commit messages, it generates more pings over time)

@fakenine fakenine force-pushed the normalize_use_of_backticks_compiler_messages_p2 branch from 85f1888 to 60a4938 Compare July 7, 2019 17:23
@fakenine
Copy link
Contributor Author

fakenine commented Jul 7, 2019

@Centril Sure! Done for the first commit

@Centril
Copy link
Contributor

Centril commented Jul 7, 2019

Thanks! @bors r+ rollup

@bors
Copy link
Collaborator

bors commented Jul 7, 2019

📌 Commit 60a4938 has been approved by Centril

@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 Jul 7, 2019
Centril added a commit to Centril/rust that referenced this pull request Jul 8, 2019
…_compiler_messages_p2, r=Centril

Normalize use of backticks in compiler messages p2

normalize use of backticks in compiler messages for librustc_typecheck

rust-lang#60532
Centril added a commit to Centril/rust that referenced this pull request Jul 8, 2019
…_compiler_messages_p2, r=Centril

Normalize use of backticks in compiler messages p2

normalize use of backticks in compiler messages for librustc_typecheck

rust-lang#60532
bors added a commit that referenced this pull request Jul 8, 2019
Rollup of 5 pull requests

Successful merges:

 - #62356 (Implement Option::contains and Result::contains)
 - #62462 (Document `while` keyword)
 - #62472 (Normalize use of backticks in compiler messages p2)
 - #62477 (Re-add bootstrap attribute to libunwind for llvm-libunwind feature)
 - #62478 (normalize use of backticks for compiler messages in librustc_codegen)

Failed merges:

r? @ghost
@bors bors merged commit 60a4938 into rust-lang:master Jul 8, 2019
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.

4 participants