Skip to content

Revert "Raise the minimum deployment target to 10.11 (#3384)" #3388

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
Apr 7, 2021

Conversation

neonichu
Copy link
Contributor

@neonichu neonichu commented Apr 6, 2021

This reverts commit 5fe380c.

With swiftlang/swift-driver#581 being merged, we can undo this temporary workaround.

rdar://76167831

This reverts commit 5fe380c.

With swiftlang/swift-driver#581 being merged, we can undo this temporary workaround.

rdar://76167831
@neonichu
Copy link
Contributor Author

neonichu commented Apr 6, 2021

@benlangmuir I think this will mean SourceKit-LSP tests need to change again, I can open a PR for that. I'm assuming if we actually run the full smoke tests, this will be uncovered as a failure, right?

@neonichu
Copy link
Contributor Author

neonichu commented Apr 6, 2021

@swift-ci please smoke test

@benlangmuir
Copy link
Contributor

@neonichu I think sourcekit-lsp will be fine this time, because I updated our tests to use Platform.macOS.oldestSupportedVersion instead of hard coding the version string (swiftlang/sourcekit-lsp#385). But yeah, as long as we wait for all the PR tests it should catch any issues 👍

@neonichu neonichu merged commit b94b0bc into swiftlang:main Apr 7, 2021
@neonichu neonichu deleted the revert-deployment-target-change branch April 7, 2021 17:54
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