Skip to content

Add PHP 8.3 to test matrix#245

Merged
brettmc merged 4 commits into
open-telemetry:mainfrom
agoallikmaa:ci-test-83
Mar 12, 2024
Merged

Add PHP 8.3 to test matrix#245
brettmc merged 4 commits into
open-telemetry:mainfrom
agoallikmaa:ci-test-83

Conversation

@agoallikmaa

@agoallikmaa agoallikmaa commented Mar 11, 2024

Copy link
Copy Markdown
Contributor

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

codecov Bot commented Mar 11, 2024

Copy link
Copy Markdown

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