Skip to content

Make test err annotations formatting uniform #103014

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

Closed
wants to merge 1 commit into from
Closed

Make test err annotations formatting uniform #103014

wants to merge 1 commit into from

Conversation

Rageking8
Copy link
Contributor

Convert all test err annotation formatting to the style within rustc dev guide.

I have manually checked every single line of diff to ensure that no unwanted changes slip through. I excluded anything under src/tools. Thanks.

@rustbot rustbot added T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. labels Oct 13, 2022
@rust-highfive
Copy link
Contributor

r? @Mark-Simulacrum

(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 Oct 13, 2022
@Mark-Simulacrum
Copy link
Member

Mark-Simulacrum commented Oct 16, 2022

I am not sure this is worth doing - we parse both formats in compiletest, and either is readable, so normalizing them only adds a speed bump when adding new definitions (presuming we enforce it) without major benefit I think.

Can you say more about what motivated this PR?

@Rageking8
Copy link
Contributor Author

I am not sure this is worth doing - we parse both formats in compiletest, and either is readable, so normalizing them only adds a speed bump when adding new definitions (presuming we enforce it) without major benefit I think.

Can you say more about what motivated this PR?

I don't think we should enforce it, it's just that when writing tests I see 2 different ways of formatting the err annotations. So, I decided to whip up a PR to clean all up with the space as with the examples shown in the aforementioned rustc dev guide. All in all, not really a really strong reason to do so but I prefer it with the space together with uniformity. No strong objections if this is just not worth it, and I can close this PR, thanks.

@Mark-Simulacrum
Copy link
Member

I think the cost in both review time (for this PR and future similar PRs) and in churn to git blame isn't worth the very slight delta in possible correlation with the dev guide. I'd be happy to see a PR to the dev-guide noting both work though.

@bors
Copy link
Collaborator

bors commented Oct 17, 2022

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

@Mark-Simulacrum Mark-Simulacrum 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-review Status: Awaiting review from the assignee but also interested parties. labels Oct 18, 2022
@Rageking8 Rageking8 closed this Oct 21, 2022
@Rageking8 Rageking8 deleted the make-test-err-annotations-formatting-uniform branch October 21, 2022 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants