Skip to content

Conversation

@pdelewski
Copy link
Member

@pdelewski pdelewski commented Feb 9, 2023

Added hooks for:

  • beginTransaction
  • commit
  • rollback
  • prepare

Removed fetch for now. Extended tests.

@pdelewski pdelewski requested a review from a team February 9, 2023 15:47
@codecov
Copy link

codecov bot commented Feb 9, 2023

Codecov Report

Merging #109 (8061b26) into main (d03edfb) will not change coverage.
The diff coverage is n/a.

❗ Current head 8061b26 differs from pull request most recent head 544c978. Consider uploading reports for the commit 544c978 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##               main     #109   +/-   ##
=========================================
  Coverage     64.42%   64.42%           
  Complexity      399      399           
=========================================
  Files            42       42           
  Lines          1380     1380           
=========================================
  Hits            889      889           
  Misses          491      491           
Flag Coverage Δ
7.4 81.76% <ø> (ø)
8.0 64.24% <ø> (ø)
8.1 64.32% <ø> (ø)
8.2 64.32% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d03edfb...544c978. Read the comment docs.

@pdelewski pdelewski merged commit 9c7b552 into open-telemetry:main Feb 10, 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.

2 participants