Uh oh!
There was an error while loading. Please reload this page.
Address inconsistency in using "is" with "declared here" - #68080
Conversation
Centril
commented
Jan 10, 2020
Seems like you didn't get all of them, e.g. in |
varkor
commented
Jan 10, 2020
Fixed. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Centril
commented
Jan 10, 2020
@bors r+ |
bors
commented
Jan 10, 2020
📌 Commit 1b5ad88726127be01d6f0dab013091862e4d337d has been approved by |
bors
commented
Jan 10, 2020
☔ The latest upstream changes (presumably #68101) made this pull request unmergeable. Please resolve the merge conflicts. |
varkor
commented
Jan 11, 2020
@bors r=Centril |
bors
commented
Jan 11, 2020
📌 Commit 41d0fee2453fbdbe85ed41c10be7981a0381bcf4 has been approved by |
petrochenkov
commented
Jan 11, 2020
I assumed the nominal sentences in labels are by design. I used them multiple times and @estebank used this style many times as well, if I remember correctly. (None of us is a native English speaker though.) (I agree that in primary error messages or notes full sentences seem more appropriate.) |
Centril
commented
Jan 11, 2020
I agree with this rationale but I don't care strongly. Mostly I think it is important to be consistent, in some direction.
(My neither, although @varkor is.) |
petrochenkov
commented
Jan 25, 2020
The change in "lint level defined here" breaks UI tests in clippy and it's a no tool breakage week. We can wait or remove this part from the PR and land it later. |
varkor
commented
Jan 25, 2020
Let's just wait until the branch off. |
petrochenkov
commented
Jan 30, 2020
@bors r+ |
bors
commented
Jan 30, 2020
💡 This pull request was already approved, no need to approve it again.
|
bors
commented
Jan 30, 2020
📌 Commit 4583283 has been approved by |
bors
commented
Jan 31, 2020
Address inconsistency in using "is" with "declared here" "is" was generally used for NLL diagnostics, but not other diagnostics. Using "is" makes the diagnostics sound more natural and readable, so it seems sensible to commit to them throughout. r? @Centril
rust-highfive
commented
Jan 31, 2020
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
bors
commented
Jan 31, 2020
💔 Test failed - checks-azure |
varkor
commented
Jan 31, 2020
Looks spurious to me? @bors retry |
bors
commented
Jan 31, 2020
Address inconsistency in using "is" with "declared here" "is" was generally used for NLL diagnostics, but not other diagnostics. Using "is" makes the diagnostics sound more natural and readable, so it seems sensible to commit to them throughout. r? @Centril
bors
commented
Jan 31, 2020
☀️ Test successful - checks-azure |
rust-highfive
commented
Jan 31, 2020
📣 Toolstate changed by #68080! Tested on commit 5371ddf. 💔 clippy-driver on windows: test-pass → test-fail (cc @mcarton@oli-obk@Manishearth@flip1995@yaahc@phansch@llogiq, @rust-lang/infra). |
"is" was generally used for NLL diagnostics, but not other diagnostics. Using "is" makes the diagnostics sound more natural and readable, so it seems sensible to commit to them throughout.
r? @Centril