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

PostgresClient implements ServiceLifecycle's Service #457

Merged

Conversation

fabianfett
Copy link
Collaborator

PostgresClient should conform to ServiceLifecycle's Service to play nicely with the ecosystem.

@fabianfett fabianfett added the semver-minor Adds new public API. label Feb 21, 2024
@gwynne
Copy link
Member

gwynne commented Feb 21, 2024

I don't see any actual conformance here?

@fabianfett fabianfett force-pushed the ff-PostgresClient-implements-ServiceLifecycle branch from 47c215e to 2eb5334 Compare February 21, 2024 16:23
@codecov-commenter
Copy link

codecov-commenter commented Feb 21, 2024

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (17b23b1) 61.82% compared to head (725ac1d) 61.81%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #457      +/-   ##
==========================================
- Coverage   61.82%   61.81%   -0.02%     
==========================================
  Files         125      125              
  Lines       10012    10014       +2     
==========================================
  Hits         6190     6190              
- Misses       3822     3824       +2     
Files Coverage Δ
Sources/PostgresNIO/Pool/PostgresClient.swift 0.00% <0.00%> (ø)

@fabianfett fabianfett merged commit c75349f into vapor:main Feb 22, 2024
12 of 13 checks passed
@fabianfett fabianfett deleted the ff-PostgresClient-implements-ServiceLifecycle branch February 22, 2024 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver-minor Adds new public API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants