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

chore: makes sourcery buildable starting from Xcode 13.0 / Swift 5.5 (#1045) #1046

Closed
wants to merge 1 commit into from
Closed

chore: makes sourcery buildable starting from Xcode 13.0 / Swift 5.5 (#1045) #1046

wants to merge 1 commit into from

Conversation

AF-cgi
Copy link

@AF-cgi AF-cgi commented Mar 31, 2022

Solves the issue of #1045. Further the size of the build artifact has been reduced dramatically. Now Sourcery is buildable starting from Xcode 13.0.

@krzysztofzablocki
Copy link
Owner

This isn't correct, you are removing dependency on libSwiftSyntax and the work that @liamnichols did to make it static instead of dynamically linked, this will break not only Mint support but all of the versions, it will fail or work depending on what Xcode people have on their machines.

@liamnichols
Copy link
Collaborator

If we do want to go ahead with this, we'd want to keep using StaticInternalSwiftSyntaxParser, which can be done relatively simply ... I had a go in #1049 but there are a few things to think about (see the PR description)

1 similar comment
@liamnichols
Copy link
Collaborator

If we do want to go ahead with this, we'd want to keep using StaticInternalSwiftSyntaxParser, which can be done relatively simply ... I had a go in #1049 but there are a few things to think about (see the PR description)

@AF-cgi AF-cgi closed this Apr 5, 2022
@AF-cgi AF-cgi deleted the maint/improve-swift-syntax branch April 5, 2022 12:00
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