Skip to content

Conversation

@vbergeron
Copy link

Previous iteration on the Session API were mising the .stream and
.cursor methods from the PreparedQuery delegation.

I also cleaned up the type parameter on .transaction since it was never used.

@codecov
Copy link

codecov bot commented Jan 4, 2023

Codecov Report

Merging #774 (55c0f9b) into main (9a4d320) will decrease coverage by 0.05%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##             main     #774      +/-   ##
==========================================
- Coverage   84.89%   84.84%   -0.06%     
==========================================
  Files         124      124              
  Lines        1668     1669       +1     
  Branches      109      117       +8     
==========================================
  Hits         1416     1416              
- Misses        252      253       +1     
Impacted Files Coverage Δ
modules/core/shared/src/main/scala/Session.scala 56.96% <33.33%> (-0.74%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@vbergeron vbergeron force-pushed the session-api branch 2 times, most recently from 282291d to 7f468c3 Compare January 6, 2023 15:00
Previous iteration on the Session API were mising the `.stream` and
 `.cursor` methods from the PreparedQuery delegation.
@mpilquist mpilquist merged commit 9f71412 into typelevel:main Jan 17, 2023
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.

3 participants