Skip to content

Drop support for Swift 5.5 #336

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 2 commits into from
Apr 14, 2023
Merged

Drop support for Swift 5.5 #336

merged 2 commits into from
Apr 14, 2023

Conversation

fabianfett
Copy link
Collaborator

Motivation

@gwynne started to drop Swift 5.5 in #335. However a lot of code that we can remove now wasn't touched there. And it should not be touched in there. This PR removes all code that we can drop when requiring Swift 5.6.

Changes

  • Cleanup code after dropping 5.5

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 merge this after #335 once CI is updated

Copy link
Member

@gwynne gwynne left a comment

Choose a reason for hiding this comment

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

LOL I already have about half of this done locally (including an update for the CI), let's get this merged and I'll rebase the UDS/Channel branch from that point.

@codecov-commenter
Copy link

codecov-commenter commented Apr 14, 2023

Codecov Report

Merging #336 (e437958) into main (cf09800) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #336      +/-   ##
==========================================
- Coverage   41.61%   41.57%   -0.04%     
==========================================
  Files         115      115              
  Lines        9440     9434       -6     
==========================================
- Hits         3928     3922       -6     
  Misses       5512     5512              
Impacted Files Coverage Δ
...es/PostgresNIO/Connection/PostgresConnection.swift 17.32% <ø> (ø)
Sources/PostgresNIO/Data/PostgresData.swift 6.89% <ø> (ø)
Sources/PostgresNIO/Data/PostgresDataType.swift 5.66% <ø> (ø)
Sources/PostgresNIO/Data/PostgresRow.swift 49.24% <ø> (ø)
Sources/PostgresNIO/New/Messages/DataRow.swift 95.38% <ø> (ø)
...rces/PostgresNIO/New/Messages/RowDescription.swift 100.00% <ø> (ø)
Sources/PostgresNIO/New/PostgresCell.swift 91.17% <ø> (ø)
Sources/PostgresNIO/New/PostgresQuery.swift 75.00% <ø> (ø)
...gresNIO/New/PostgresRowSequence-multi-decode.swift 0.00% <ø> (ø)
Sources/PostgresNIO/New/PostgresRowSequence.swift 89.23% <ø> (ø)
... and 1 more

@fabianfett
Copy link
Collaborator Author

nightly crashes the compiler... ignore.

@0xTim
Copy link
Member

0xTim commented Apr 14, 2023

Yeah this is being tracked at swiftlang/swift#65064 (assuming it's the same as all the other nightly crashes we're seeing across Vapor)

@fabianfett fabianfett merged commit cf62abc into vapor:main Apr 14, 2023
@fabianfett fabianfett deleted the ff-drop-5.5 branch April 14, 2023 10:10
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.

4 participants