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

Improve attribute message error spans #79509

Merged
merged 3 commits into from
Dec 2, 2020

Conversation

GuillaumeGomez
Copy link
Member

I got the idea while working on #79464

@rust-highfive
Copy link
Collaborator

r? @eddyb

(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 Nov 28, 2020
@GuillaumeGomez GuillaumeGomez force-pushed the improve-attr-spans branch 3 times, most recently from d80616d to a264d5d Compare December 1, 2020 13:44
@oli-obk oli-obk added the A-diagnostics Area: Messages for errors, warnings, and lints label Dec 1, 2020
@oli-obk
Copy link
Contributor

oli-obk commented Dec 1, 2020

r? @oli-obk

@rust-highfive rust-highfive assigned oli-obk and unassigned eddyb Dec 1, 2020
@GuillaumeGomez GuillaumeGomez force-pushed the improve-attr-spans branch 2 times, most recently from 4d4f36d to e26914b Compare December 1, 2020 15:16
@GuillaumeGomez
Copy link
Member Author

As shown here, it was actually needed. Putting it back then haha.

@oli-obk
Copy link
Contributor

oli-obk commented Dec 1, 2020

@bors r+

@bors
Copy link
Contributor

bors commented Dec 1, 2020

📌 Commit a2d1254 has been approved by oli-obk

@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 Dec 1, 2020
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Dec 1, 2020
…r=oli-obk

Improve attribute message error spans

I got the idea while working on rust-lang#79464
bors added a commit to rust-lang-ci/rust that referenced this pull request Dec 1, 2020
…laumeGomez

Rollup of 3 pull requests

Successful merges:

 - rust-lang#79508 (Warn if `dsymutil` returns an error code)
 - rust-lang#79509 (Improve attribute message error spans)
 - rust-lang#79600 (std::io: Use sendfile for UnixStream)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 3d631b0 into rust-lang:master Dec 2, 2020
@rustbot rustbot added this to the 1.50.0 milestone Dec 2, 2020
@GuillaumeGomez GuillaumeGomez deleted the improve-attr-spans branch December 2, 2020 09:15
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 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.

6 participants