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 12 pull requests #35652

Closed
wants to merge 28 commits into from
Closed

Rollup of 12 pull requests #35652

wants to merge 28 commits into from

Conversation

stanislav-tkach and others added 27 commits August 9, 2016 13:13
E0248 Change in issue format

E0267 UT New Format

E0268 UT New Format

E0267 & E0268 New Error Format
changed error text
…handturner

Update compiler error 0093 to use new error format

Addresses rust-lang#35230, Part of the rust-lang#35233 meta bug

r? @jonathandturner
Update E0220 message to new format

Part of rust-lang#35233 .
Fixes rust-lang#35385.

r? @jonathandturner

Should it keep E0191?
Update E0138 to new format

Part of rust-lang#35233
Fix rust-lang#35510
r? @jonathandturner

![e0138](https://cloud.githubusercontent.com/assets/2716047/17562415/7200d93c-5f5d-11e6-98ff-e15c29f40e03.png)

Question: How can I only underline the function name ? I have observed the debug output and the struct of item, but I can't find the `Span` for function name. Should I modify the struct I get to save function name's position or there is another way to get it ? (I can only find `Span`s for function attributes, inputs, outputs, blocks)
…turner

E0248, E0267 & E0268 Change into issue format

r? @jonathandturner  Part of rust-lang#35391, rust-lang#35519 and rust-lang#35520. I have squashed all changes into a single commit. Please review the changes.

E0248 Change in issue format

E0267 UT New Format

E0268 UT New Format

E0267 & E0268 New Error Format
…ts, r=jonathandturner

Add label to E0254

This issue rust-lang#35513 is a part of rust-lang#35233.
r? @jonathandturner
… r=jonathandturner

Update E0070 to new error format

Updated E0070 to new error format.
Part of rust-lang#35233
Fixes rust-lang#35503

Thanks for letting me help!

r? @jonathandturner
… r=jonathandturner

changed E0067 to new error format

Updated E0067 to new error format.
Part of rust-lang#35233
Fixes rust-lang#35502

Passes all the tests when running:
`python src/bootstrap/bootstrap.py --step check-cfail --stage 1`

**This seems strange, given that the format for E0067 has been changed.**
It feels like it should fail some unit tests maybe?

Let me know if I'm mistaken. Otherwise I can create a unit test for it.

Thanks for letting me help!

r? @jonathandturner
…ormat, r=jonathandturner

Update E0301 to the new format

Part of rust-lang#35233.
Fixes rust-lang#35522.

r? @jonathandturner
…ormat, r=jonathandturner

Update E0302 to the new format

Part of rust-lang#35233.
Fixes rust-lang#35523.

r? @jonathandturner
@rust-highfive
Copy link
Collaborator

r? @eddyb

(rust_highfive has picked a reviewer for you, use r? to override)

@sophiajt
Copy link
Contributor Author

@bors r+ p=10

@bors
Copy link
Contributor

bors commented Aug 13, 2016

📌 Commit 913d2f7 has been approved by jonathandturner

@sophiajt
Copy link
Contributor Author

@bors r+ p=10

@bors
Copy link
Contributor

bors commented Aug 13, 2016

📌 Commit 5e9dc84 has been approved by jonathandturner

@bors
Copy link
Contributor

bors commented Aug 13, 2016

⌛ Testing commit 5e9dc84 with merge 1ce37cd...

@bors
Copy link
Contributor

bors commented Aug 13, 2016

💔 Test failed - auto-win-gnu-32-opt-rustbuild

@bors
Copy link
Contributor

bors commented Aug 15, 2016

☔ The latest upstream changes (presumably #35666) made this pull request unmergeable. Please resolve the merge conflicts.

@eddyb eddyb closed this Aug 15, 2016
@Centril Centril added the rollup A PR which is a rollup label Oct 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup
Projects
None yet
Development

Successfully merging this pull request may close these issues.