Skip to content
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

Run CI with Xcode 12.5.1 & Swift 5.4 #3710

Merged
merged 2 commits into from
Sep 10, 2021
Merged

Conversation

jpsim
Copy link
Collaborator

@jpsim jpsim commented Sep 9, 2021

And require Swift 5.4 or Xcode 12.5 or later to build SwiftLint.

@jpsim jpsim marked this pull request as ready for review September 9, 2021 21:04
@jpsim jpsim force-pushed the run-ci-with-xcode-12.5.1--swift-5.4 branch from c98d2d1 to 4755cf1 Compare September 9, 2021 21:11
@SwiftLintBot
Copy link

SwiftLintBot commented Sep 9, 2021

12 Messages
📖 Linting Aerial with this PR took 0.73s vs 0.73s on master (0% slower)
📖 Linting Alamofire with this PR took 0.96s vs 0.95s on master (1% slower)
📖 Linting Firefox with this PR took 3.41s vs 3.41s on master (0% slower)
📖 Linting Kickstarter with this PR took 6.97s vs 6.96s on master (0% slower)
📖 Linting Moya with this PR took 0.5s vs 0.5s on master (0% slower)
📖 Linting Nimble with this PR took 0.4s vs 0.41s on master (2% faster)
📖 Linting Quick with this PR took 0.19s vs 0.19s on master (0% slower)
📖 Linting Realm with this PR took 2.24s vs 2.24s on master (0% slower)
📖 Linting SourceKitten with this PR took 0.33s vs 0.33s on master (0% slower)
📖 Linting Sourcery with this PR took 1.95s vs 1.95s on master (0% slower)
📖 Linting Swift with this PR took 3.15s vs 3.15s on master (0% slower)
📖 Linting WordPress with this PR took 6.34s vs 6.33s on master (0% slower)

Generated by 🚫 Danger

Because the Azure Pipelines [macos-11](https://github.com/actions/virtual-environments/blob/main/images/macos/macos-11-Readme.md)
image has fewer Xcode versions available, we'll only continue testing
Swift 5.3 support on Linux, which I think is a fine safety net
personally and not worth duplicating some CI jobs to run on macos-10.15
for access to Xcode 12.0.
@jpsim jpsim force-pushed the run-ci-with-xcode-12.5.1--swift-5.4 branch from af40b10 to 1eacdc9 Compare September 10, 2021 11:20
@jpsim jpsim merged commit 8886199 into master Sep 10, 2021
@jpsim jpsim deleted the run-ci-with-xcode-12.5.1--swift-5.4 branch September 10, 2021 15:32
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