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

Show duplicate diagnostics in UI tests by default #12374

Merged
merged 1 commit into from
Feb 28, 2024

Conversation

Alexendoo
Copy link
Member

Duplicated diagnostics can indicate where redundant work is being done, this PR doesn't fix any of that but does indicate in which tests they're occurring for future investigation or to catch issues in future lints

changelog: none

@rustbot
Copy link
Collaborator

rustbot commented Feb 28, 2024

r? @Manishearth

rustbot has assigned @Manishearth.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Feb 28, 2024
@Alexendoo Alexendoo force-pushed the duplicate-diagnostics branch from cefc481 to ae08cd2 Compare February 28, 2024 13:24
@Alexendoo Alexendoo force-pushed the duplicate-diagnostics branch from ae08cd2 to 733e1d4 Compare February 28, 2024 13:24
@Alexendoo Alexendoo changed the title Do not deduplicate diagnostics in UI tests by default Show duplicate diagnostics in UI tests by default Feb 28, 2024
@Manishearth
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Feb 28, 2024

📌 Commit 733e1d4 has been approved by Manishearth

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Feb 28, 2024

⌛ Testing commit 733e1d4 with merge af91e6e...

@bors
Copy link
Contributor

bors commented Feb 28, 2024

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: Manishearth
Pushing af91e6e to master...

@bors bors merged commit af91e6e into rust-lang:master Feb 28, 2024
5 checks passed
@flip1995
Copy link
Member

#12379

Opened an issue, so that the current tests might also get cleaned up over time.

@Alexendoo Alexendoo deleted the duplicate-diagnostics branch February 28, 2024 17:09
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.

5 participants