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

Update for Swift 5.10 and strict concurrency #125

Closed
wants to merge 6 commits into from
Closed

Conversation

gwynne
Copy link
Member

@gwynne gwynne commented Mar 10, 2024

Updates the template for Swift 5.10, removes strict concurrency warnings, and enables them by default. This PR is an extended version of the work started in #124.

Shoutout and thanks to @ameter for the initial work on this!

@gwynne gwynne added the enhancement New feature or request label Mar 10, 2024
@gwynne gwynne requested review from 0xTim, MahdiBM and ptoffy March 10, 2024 19:53
@0xTim
Copy link
Member

0xTim commented Mar 10, 2024

See https://github.com/vapor/template/pull/124/files#r1519002773 - we cannot add @Sendable to functions

.target(name: "App"),
.product(name: "XCTVapor", package: "vapor"),

// Workaround for https://github.com/apple/swift-package-manager/issues/6940
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this not fixed on 5.10?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe so, yes

Copy link
Member

@0xTim 0xTim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can remove the manifest workaround as that works in 5.10

.target(name: "App"),
.product(name: "XCTVapor", package: "vapor"),

// Workaround for https://github.com/apple/swift-package-manager/issues/6940
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe so, yes

@0xTim
Copy link
Member

0xTim commented Apr 9, 2024

Closing in favour of #127

@0xTim 0xTim closed this Apr 9, 2024
@gwynne gwynne deleted the swift-5.10-update branch May 7, 2024 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants