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

fix: Only show unlinked-file diagnostic on first line during startup #17415

Merged
merged 1 commit into from
Jun 19, 2024

Conversation

Wilfred
Copy link
Contributor

@Wilfred Wilfred commented Jun 13, 2024

This partially reverts #17350, based on the feedback in #17397.

If we don't have an autofix, it's more annoying to highlight the whole file. This autofix heuristic fixes the diagnostic being overwhelming during startup.

This partially reverts rust-lang#17350, based on the feedback in rust-lang#17397.

If we don't have an autofix, it's more annoying to highlight the whole line.
This heuristic fixes the diagnostic overwhelming the user during startup.
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 13, 2024
@Wilfred Wilfred marked this pull request as ready for review June 13, 2024 18:55
@Veykril
Copy link
Member

Veykril commented Jun 19, 2024

Thanks!
@bors r+

@bors
Copy link
Contributor

bors commented Jun 19, 2024

📌 Commit 495c0cb has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Jun 19, 2024

⌛ Testing commit 495c0cb with merge 889ca28...

@bors
Copy link
Contributor

bors commented Jun 19, 2024

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing 889ca28 to master...

@bors bors merged commit 889ca28 into rust-lang:master Jun 19, 2024
11 checks passed
@Wilfred Wilfred deleted the unlinked_diagnostic_span branch August 19, 2024 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants