Skip to content

Conversation

camelid
Copy link
Member

@camelid camelid commented Jan 8, 2021

Nearly all error messages start with a lowercase letter and don't use
articles - instead they refer to the plural case.

@camelid camelid added A-diagnostics Area: Messages for errors, warnings, and lints D-inconsistent Diagnostics: Inconsistency in formatting, grammar or style between diagnostic messages. labels Jan 8, 2021
@rust-highfive
Copy link
Contributor

r? @oli-obk

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 8, 2021
@lcnr
Copy link
Contributor

lcnr commented Jan 8, 2021

r? @lcnr
@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Jan 8, 2021

📌 Commit 4e3be7c has been approved by lcnr

@rust-highfive rust-highfive assigned lcnr and unassigned oli-obk Jan 8, 2021
@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 Jan 8, 2021
JohnTitor added a commit to JohnTitor/rust that referenced this pull request Jan 10, 2021
Use standard formatting for "rust-call" ABI message

Nearly all error messages start with a lowercase letter and don't use
articles - instead they refer to the plural case.
@JohnTitor
Copy link
Member

Failed in rollup: https://github.com/rust-lang/rust/runs/1676010238
Should rebase and bless the tests.

@bors r-

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jan 10, 2021
Nearly all error messages start with a lowercase letter and don't use
articles - instead they refer to the plural case.
@camelid camelid added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jan 10, 2021
@camelid
Copy link
Member Author

camelid commented Jan 10, 2021

Ready for re-approval.

@lcnr
Copy link
Contributor

lcnr commented Jan 10, 2021

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Jan 10, 2021

📌 Commit 6488aec has been approved by lcnr

@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 Jan 10, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Jan 11, 2021
Rollup of 6 pull requests

Successful merges:

 - rust-lang#80809 (Use standard formatting for "rust-call" ABI message)
 - rust-lang#80872 (Fix typo in source-based-code-coverage.md)
 - rust-lang#80878 (Add ABI argument to `find_mir_or_eval_fn`)
 - rust-lang#80881 ( Fix intra-doc links to `Self` and `crate` )
 - rust-lang#80887 (log-color: Detect TTY based on stderr, not stdout)
 - rust-lang#80892 (rustdoc: Remove `*` intra-doc alias for `pointer`)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit dcd46bf into rust-lang:master Jan 11, 2021
@rustbot rustbot added this to the 1.51.0 milestone Jan 11, 2021
@bors
Copy link
Collaborator

bors commented Jan 11, 2021

⌛ Testing commit 6488aec with merge a2cd91c...

@camelid camelid deleted the rust-call-abi-msg branch January 11, 2021 21:17
@camelid
Copy link
Member Author

camelid commented Jan 11, 2021

Heh, looks like there was a race between the rollup merge and bors testing 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-diagnostics Area: Messages for errors, warnings, and lints D-inconsistent Diagnostics: Inconsistency in formatting, grammar or style between diagnostic messages. 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.

7 participants