Skip to content

Conversation

@ahoppen
Copy link
Member

@ahoppen ahoppen commented May 25, 2024

Diagnostics are usually not helpful if the file’s target hasn’t been prepared yet. We should await the target’s preparation before returning diagnostics.

rdar://128645617

@ahoppen ahoppen requested review from bnbarham and hamishknight May 25, 2024 16:40
@ahoppen ahoppen requested a review from benlangmuir as a code owner May 25, 2024 16:40
@ahoppen
Copy link
Member Author

ahoppen commented May 25, 2024

@swift-ci Please test

@ahoppen ahoppen force-pushed the wait-for-preparation-before-diags branch from 6eb47d3 to d18e1b7 Compare May 25, 2024 20:51
@ahoppen
Copy link
Member Author

ahoppen commented May 25, 2024

@swift-ci Please test

Diagnostics are usually not helpful if the file’s target hasn’t been prepared yet. We should await the target’s preparation before returning diagnostics.

rdar://128645617
@ahoppen ahoppen force-pushed the wait-for-preparation-before-diags branch from d18e1b7 to 05dd102 Compare May 27, 2024 01:55
@ahoppen
Copy link
Member Author

ahoppen commented May 27, 2024

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented May 27, 2024

@swift-ci Please test Windows

@ahoppen ahoppen merged commit a29f1ac into swiftlang:main May 28, 2024
@ahoppen ahoppen deleted the wait-for-preparation-before-diags branch August 4, 2025 10:48
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.

2 participants