Skip to content

Conversation

@agoallikmaa
Copy link
Contributor

@agoallikmaa agoallikmaa commented Mar 11, 2024

Add PHP 8.3 to test matrix. Did not test everything locally on 8.3 beforehand, so will keep as draft until any issues revealed by CI are fixed.

Updated psalm as the previous version of either psalm itself or its phpunit plugin did not support 8.3. This required some adjustment of psalm rules in some code.

@codecov
Copy link

codecov bot commented Mar 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.47%. Comparing base (a7ff7db) to head (1cd687a).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##               main     #245   +/-   ##
=========================================
  Coverage     85.47%   85.47%           
  Complexity      972      972           
=========================================
  Files            88       88           
  Lines          3923     3923           
=========================================
  Hits           3353     3353           
  Misses          570      570           
Flag Coverage Δ
7.4 85.72% <ø> (ø)
8.0 84.61% <ø> (ø)
8.1 84.72% <ø> (ø)
8.2 85.39% <ø> (ø)
8.3 85.39% <ø> (?)

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

Files Coverage Δ
.../Instrumentation/Laravel/src/HeadersPropagator.php 42.85% <ø> (ø)
...trumentation/Laravel/src/Watchers/QueryWatcher.php 95.65% <ø> (ø)
...n/MongoDB/src/MongoDBInstrumentationSubscriber.php 84.37% <ø> (ø)
...rc/Instrumentation/PDO/src/PDOAttributeTracker.php 96.55% <ø> (ø)
...mentation/Symfony/src/RequestPropagationGetter.php 71.42% <ø> (ø)
src/Shims/OpenTracing/src/Span.php 93.47% <ø> (ø)
...c/OtelBundle/DependencyInjection/Configuration.php 100.00% <ø> (ø)
src/Symfony/src/OtelSdkBundle/Util/ExporterDsn.php 100.00% <ø> (ø)

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 a7ff7db...1cd687a. Read the comment docs.

@agoallikmaa agoallikmaa marked this pull request as ready for review March 11, 2024 07:31
@agoallikmaa agoallikmaa requested a review from a team March 11, 2024 07:31
@brettmc brettmc merged commit 1e30145 into open-telemetry:main Mar 12, 2024
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