Skip to content

Add another PostgresClient API - #231

Merged
ptoffy merged 3 commits into
mainfrom
postgres-nio-again
Sep 28, 2026
Merged

ptoffy merged 3 commits into
mainfrom
postgres-nio-again

Conversation

@ptoffy

@ptoffy ptoffy commented Sep 24, 2026 •

Copy link
Copy Markdown
Member

These changes are now available in 2.14.0

If people have their own managed PostgresClient they should be able to use it

@ptoffy
ptoffy requested a review from gwynne as a code owner September 24, 2026 09:05
@gwynne gwynne added the semver-minor Contains new APIs label Sep 24, 2026

@gwynne gwynne left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

S'long as it works, looks fine to me.

@codecov

codecov Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 92.85714% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.36%. Comparing base (b5d2314) to head (517361b).

Files with missing lines Patch % Lines
...entPostgresDriver/FluentPostgresClientDriver.swift 78.57% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #231      +/-   ##
==========================================
+ Coverage   59.79%   64.36%   +4.57%     
==========================================
  Files          12       12              
  Lines         582      609      +27     
==========================================
+ Hits          348      392      +44     
+ Misses        234      217      -17     
Files with missing lines Coverage Δ
...gresDriver/FluentPostgresClientConfiguration.swift 100.00% <100.00%> (ø)
...entPostgresDriver/FluentPostgresClientDriver.swift 87.50% <78.57%> (ø)

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@0xTim 0xTim left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

On mobile but LGTM

@ptoffy
ptoffy merged commit 26fe819 into main Sep 28, 2026
10 checks passed
@ptoffy
ptoffy deleted the postgres-nio-again branch September 28, 2026 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver-minor Contains new APIs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants