Skip to content

Conversation

gwynne
Copy link
Member

@gwynne gwynne commented Aug 25, 2025

These changes are now available in 2.14.1

Bump Swift min to 5.10, update README and API docs, clean up some doc comments, run swift-format, fix warning about retroactive conformance in 6.2.

@gwynne gwynne requested review from 0xTim, MahdiBM and ptoffy August 25, 2025 10:29
@gwynne gwynne added the semver-patch Internal changes only label Aug 25, 2025
Copy link

codecov bot commented Aug 25, 2025

Codecov Report

❌ Patch coverage is 96.42857% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 42.47%. Comparing base (a4212e1) to head (93cac3b).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
Sources/PostgresKit/PostgresDataTranslation.swift 92.30% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #273      +/-   ##
==========================================
+ Coverage   41.84%   42.47%   +0.63%     
==========================================
  Files          13       13              
  Lines         865      871       +6     
==========================================
+ Hits          362      370       +8     
+ Misses        503      501       -2     
Files with missing lines Coverage Δ
Sources/PostgresKit/PostgresDatabase+SQL.swift 76.56% <100.00%> (+1.98%) ⬆️
Sources/PostgresKit/PostgresDialect.swift 78.87% <100.00%> (+1.40%) ⬆️
Sources/PostgresKit/SQLPostgresConfiguration.swift 85.10% <ø> (ø)
Sources/PostgresKit/PostgresDataTranslation.swift 67.63% <92.30%> (+0.48%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

… comments, run swift-format, fix warning about retroactive conformance in 6.2.
Copy link
Contributor

@MahdiBM MahdiBM left a comment

Choose a reason for hiding this comment

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

some comments

@gwynne gwynne requested a review from MahdiBM August 26, 2025 09:56
@gwynne gwynne merged commit d2fd317 into main Aug 26, 2025
14 checks passed
@gwynne gwynne deleted the cleanup branch August 26, 2025 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver-patch Internal changes only
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants