Skip to content

Conversation

@ahoppen
Copy link
Member

@ahoppen ahoppen commented May 17, 2024

We could get DocumentDiagnosticsRequest triggered from file changes or old preparation operations during the initial build without actually having re-prepared the target after the file change. Make the test wait until it receives expected diagnostics after the file change.

…epareTarget`

We could get `DocumentDiagnosticsRequest` triggered from file changes or old preparation operations during the initial build without actually having re-prepared the target after the file change. Make the test wait until it receives expected diagnostics after the file change.
@ahoppen ahoppen requested review from bnbarham and hamishknight May 17, 2024 20:23
@ahoppen ahoppen requested a review from benlangmuir as a code owner May 17, 2024 20:23
@ahoppen
Copy link
Member Author

ahoppen commented May 17, 2024

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented May 17, 2024

@swift-ci Please test Windows

@ahoppen ahoppen merged commit 0ce2107 into swiftlang:main May 20, 2024
@ahoppen ahoppen deleted the prepare-target-test-failure branch May 20, 2024 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants