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

Fixed Swift major version handling #1019

Conversation

erdnussflips
Copy link

No description provided.

@erdnussflips
Copy link
Author

Why this code changes breaks the tests? Any ideas?

@johnfairh
Copy link
Collaborator

(you can see the test failure log by following the 'Details' link in the CI report)

The problem is that '4.0' is not a good default for --swift-version. This needs to stay at nil to mean whatever is configured as default for this shell.

I think the right fix here is solely to podspec_documenter and needs to pass through a specified Swift version. I think the "clip to major.0" logic comes from the Xcode 8 era when even though we were in Swift 3.2, the software required '3.0' to be passed here. Since at least Swift 4.2 this checking has got more permissive. So, taking it out would probably only break Xcode 8 users. Maybe make it conditional on version major < 4?

@johnfairh
Copy link
Collaborator

(working on fixing this up)

@johnfairh
Copy link
Collaborator

Continued in #1037.

@johnfairh johnfairh closed this Jan 12, 2019
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