Skip to content

Minor cleanups - #273

Merged
gwynne merged 3 commits into
mainfrom
cleanup
Aug 26, 2025
Merged

Minor cleanups#273
gwynne merged 3 commits into
mainfrom
cleanup

Conversation

@gwynne

@gwynne gwynne commented Aug 25, 2025

Copy link
Copy Markdown
Member

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
@codecov

codecov Bot commented Aug 25, 2025

Copy link
Copy Markdown

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.
Comment thread README.md
Comment thread README.md Outdated
Comment thread README.md
Comment thread README.md
Comment thread Sources/PostgresKit/Docs.docc/PostgresKit.md Outdated
Comment thread Sources/PostgresKit/Docs.docc/PostgresKit.md Outdated
Comment thread Sources/PostgresKit/Docs.docc/PostgresKit.md Outdated
Comment thread Sources/PostgresKit/Docs.docc/PostgresKit.md Outdated
Comment thread Sources/PostgresKit/Docs.docc/PostgresKit.md

@MahdiBM MahdiBM left a comment

Copy link
Copy Markdown
Contributor

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