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: Don't emit semantic diagnostics in files with a lot of syntax errors #17536

Merged
merged 1 commit into from
Jul 3, 2024

Conversation

Veykril
Copy link
Member

@Veykril Veykril commented Jul 3, 2024

These will only add to the noise when something very unexpected breaks or where parser recovery fails to kick in.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 3, 2024
@Veykril
Copy link
Member Author

Veykril commented Jul 3, 2024

@bors r+

@bors
Copy link
Collaborator

bors commented Jul 3, 2024

📌 Commit cbcb977 has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Jul 3, 2024

⌛ Testing commit cbcb977 with merge cae997e...

@bors
Copy link
Collaborator

bors commented Jul 3, 2024

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

@bors bors merged commit cae997e into rust-lang:master Jul 3, 2024
11 checks passed
@Veykril Veykril deleted the syntax-diags branch July 3, 2024 09:14
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.

None yet

3 participants