Skip to content

Fix psalm regressions.#1680

Merged
brettmc merged 1 commit into
open-telemetry:mainfrom
ChrisLightfootWild:fix/psalm
Aug 6, 2025
Merged

Fix psalm regressions.#1680
brettmc merged 1 commit into
open-telemetry:mainfrom
ChrisLightfootWild:fix/psalm

Conversation

@ChrisLightfootWild

Copy link
Copy Markdown
Contributor

I messed up the last rebase and broke psalm.

@ChrisLightfootWild
ChrisLightfootWild requested a review from a team as a code owner August 5, 2025 23:26
@codecov

codecov Bot commented Aug 5, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 28.57143% with 5 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (main@7981956). Learn more about missing BASE report.
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...onfiguration/Resolver/SdkConfigurationResolver.php 0.00% 2 Missing ⚠️
...tion/Environment/Adapter/SymfonyDotenvProvider.php 0.00% 1 Missing ⚠️
...on/Environment/Adapter/VlucasPhpdotenvProvider.php 0.00% 1 Missing ⚠️
...ig/SDK/Configuration/Environment/LazyEnvSource.php 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1680   +/-   ##
=======================================
  Coverage        ?   68.53%           
  Complexity      ?     2878           
=======================================
  Files           ?      430           
  Lines           ?     8759           
  Branches        ?        0           
=======================================
  Hits            ?     6003           
  Misses          ?     2756           
  Partials        ?        0           
Flag Coverage Δ
8.1 68.20% <28.57%> (?)
8.2 68.40% <28.57%> (?)
8.3 68.33% <28.57%> (?)
8.4 68.39% <28.57%> (?)
8.5 68.36% <28.57%> (?)

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

Files with missing lines Coverage Δ
...mon/Configuration/Resolver/EnvironmentResolver.php 100.00% <100.00%> (ø)
...K/Common/Configuration/Resolver/PhpIniResolver.php 100.00% <100.00%> (ø)
...tion/Environment/Adapter/SymfonyDotenvProvider.php 0.00% <0.00%> (ø)
...on/Environment/Adapter/VlucasPhpdotenvProvider.php 0.00% <0.00%> (ø)
...ig/SDK/Configuration/Environment/LazyEnvSource.php 0.00% <0.00%> (ø)
...onfiguration/Resolver/SdkConfigurationResolver.php 0.00% <0.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 7981956...ac81542. Read the comment docs.

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

@brettmc
brettmc merged commit 99a35da into open-telemetry:main Aug 6, 2025
11 checks passed
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