Skip to content

feat: sqlcommenter for PDO, MySqli, PostgreSql (pdo_mysql, pdo_pgsql)#442

Merged
brettmc merged 21 commits into
open-telemetry:mainfrom
jerrytfleung:opentelemetry-sqlcommenter
Oct 5, 2025
Merged

feat: sqlcommenter for PDO, MySqli, PostgreSql (pdo_mysql, pdo_pgsql)#442
brettmc merged 21 commits into
open-telemetry:mainfrom
jerrytfleung:opentelemetry-sqlcommenter

nits

2b2be6c
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Sep 26, 2025 in 0s

82.52% (+1.35%) compared to 1e2fa80

View this Pull Request on Codecov

82.52% (+1.35%) compared to 1e2fa80

Details

Codecov Report

❌ Patch coverage is 65.45455% with 57 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.52%. Comparing base (1e2fa80) to head (2b2be6c).
⚠️ Report is 5 commits behind head on main.

Files with missing lines Patch % Lines
src/Instrumentation/PDO/src/PDOInstrumentation.php 25.00% 30 Missing ⚠️
...strumentation/MySqli/src/MySqliInstrumentation.php 58.82% 14 Missing ⚠️
...ation/PostgreSql/src/PostgreSqlInstrumentation.php 54.54% 10 Missing ⚠️
src/SqlCommenter/src/ContextPropagatorFactory.php 93.33% 2 Missing ⚠️
src/SqlCommenter/src/SqlCommenter.php 96.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #442      +/-   ##
============================================
+ Coverage     81.17%   82.52%   +1.35%     
- Complexity     1454     2414     +960     
============================================
  Files           102      167      +65     
  Lines          5439     9711    +4272     
============================================
+ Hits           4415     8014    +3599     
- Misses         1024     1697     +673     
Files with missing lines Coverage Δ
src/SqlCommenter/src/Utils.php 100.00% <100.00%> (ø)
src/SqlCommenter/src/SqlCommenter.php 96.00% <96.00%> (ø)
src/SqlCommenter/src/ContextPropagatorFactory.php 93.33% <93.33%> (ø)
...ation/PostgreSql/src/PostgreSqlInstrumentation.php 93.95% <54.54%> (ø)
...strumentation/MySqli/src/MySqliInstrumentation.php 94.40% <58.82%> (ø)
src/Instrumentation/PDO/src/PDOInstrumentation.php 83.77% <25.00%> (-10.50%) ⬇️

... and 69 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

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

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