-
Notifications
You must be signed in to change notification settings - Fork 10.5k
AST: support availability on Windows #34439
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
Conversation
@swift-ci please test |
CC: @benrimmington |
@swift-ci please test Windows platform |
Build failed |
@swift-ci please test |
1 similar comment
@swift-ci please test |
What happens when a version number is used? The parser/runtime might allow:
|
I can add the additional test cases, however, I'm not sure that the versions make too much sense. Microsoft has announced that Windows 10 is the last version. Do we really want to have build version numbers here? I think that can get confusing and unwieldy. You would be looking at something like:
|
I was just wondering what happens when someone tries using version numbers on Windows. Does the compiler crash? Or is there an error or warning? Or are version numbers allowed at compile-time, but ignored at run-time? |
Ah, okay. I'll certainly add those test cases. It shouldn't crash the compiler, merely fail (warning/error). The version numbers aren't supported. |
@swift-ci please test |
This seems to cause some problems for SK-LSP. I want to get the reviews taken care of for the change, but leave the merging to until after the SourceKit-LSP issues have been worked out. |
@swift-ci please test |
@swift-ci please test Windows platform |
Enable Windows specific availability annotations in Swift.
406716a
to
7c968c6
Compare
@swift-ci please test |
@swift-ci please test Windows platform |
The additional tests have been added, I think that all the review comments have been addressed and SK-LSP has been updated, so optimistically merging this for now. |
Enable Windows specific availability annotations in Swift.
Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.
Resolves SR-NNNN.