-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Replace all 9999 availability in non-stdlib tests. #26109
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
@jckarter Mainly tagging you to check that you folks haven't yet added anything to these tests that actually shouldn't be available when building for 5.1, and that there aren't other reasons to maintain 9999 for these. |
@swift-ci please test |
Build failed |
Build failed |
Linux failure is in LLDB, looks unrelated to this change: 09:44:25 error: missing required modules: 'Swift', 'SwiftOnoneSupport' Modules again? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
@swift-ci smoke test linux |
Checked in with @dcci about the LLDB failures, we think this is not a PR blocker, so merging with the linux smoke-test only. |
Replace all 9999 availability in non-stdlib tests with the appropriate platform availability.
…6145) * Replace all 9999 availability in non-stdlib tests. (#26109) Replace all 9999 availability in non-stdlib tests with the appropriate platform availability. * Merge pull request #26129 from aschwaighofer/fix_test_dynamic_replacement_opaque_result Fix test/Interpreter/dynamic_replacement_opaque_result.swift
Follow-on to #26108