Skip to content

Collect diagnostics from skipped implementation files, fixes #15337 - #15351

Merged
0101 merged 2 commits into
dotnet:mainfrom
majocha:fix-missing-diag
Jun 9, 2023
Merged

Collect diagnostics from skipped implementation files, fixes #15337#15351
0101 merged 2 commits into
dotnet:mainfrom
majocha:fix-missing-diag

Conversation

@majocha

@majochamajocha commented Jun 8, 2023

Copy link
Copy Markdown
Contributor

FinalizeTypeCheckTask now collects all diagnostics, forcing type check of previously skipped implementation files.
See #15337

@majocha
majocha requested a review from a team as a code ownerJune 8, 2023 23:05
Comment threadsrc/Compiler/Service/IncrementalBuild.fs
@T-Gro

T-Gro commented Jun 9, 2023

Copy link
Copy Markdown
Member

cc @0101 for making sure this is covered in your change.

@0101

0101 commented Jun 9, 2023

Copy link
Copy Markdown
Contributor

cc @0101 for making sure this is covered in your change.

Yeah. I should probably hook it into all these tests somehow.

0101
0101 approved these changes Jun 9, 2023
@0101
0101 merged commit 8946992 into dotnet:mainJun 9, 2023
@majocha
majocha deleted the fix-missing-diag branch November 23, 2023 15:54
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants

@majocha@T-Gro@0101@nojaf@vzarytovskii