Skip to content

ConstExtract: Refactor handling of AvailabilitySpec #79449

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

Merged
merged 1 commit into from
Feb 18, 2025

Conversation

tshortli
Copy link
Contributor

Soon, AvailabilitySpec will require that the AvailabiltyDomain it contains be queried using a request that takes the DeclContext as input in order to resolve the parsed domain name to an instance of AvailabilityDomain. The constant extraction pipeline needed a bit of refactoring to thread a DeclContext through to the place where it will be needed to execute the query.

NFC.

Soon, `AvailabilitySpec` will require that the `AvailabiltyDomain` it contains
be queried using a request that takes the `DeclContext` as input in order to
resolve the parsed domain name to an instance of `AvailabilityDomain`. The
constant extraction pipeline needed a bit of refactoring to thread a
`DeclContext` through to the place where it will be needed to execute the
query.

NFC.
@tshortli
Copy link
Contributor Author

@swift-ci please smoke test

@tshortli
Copy link
Contributor Author

@swift-ci please smoke test macOS

Copy link
Contributor

@artemcm artemcm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you.

@tshortli tshortli merged commit 7aa1970 into swiftlang:main Feb 18, 2025
3 checks passed
@tshortli tshortli deleted the const-extract-availability-spec branch February 18, 2025 17:40
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