Skip to content

fixing minor static analysis complaints#1080

Merged
brettmc merged 3 commits into
open-telemetry:mainfrom
brettmc:static-analysis-fixes
Jul 19, 2023
Merged

fixing minor static analysis complaints#1080
brettmc merged 3 commits into
open-telemetry:mainfrom
brettmc:static-analysis-fixes

Conversation

@brettmc

@brettmc brettmc commented Jul 13, 2023

Copy link
Copy Markdown
Contributor

fixing static analysis complaints, most of which were from psalm when run with --show-info
static data providers (phpunit10 requirement)

fixing static analysis complaints, most of which were from psalm
when run with --show-info
remove deprecated httplug discovery
static data providers (phpunit10 requirement)
@brettmc
brettmc requested a review from a team July 13, 2023 10:35
@codecov

codecov Bot commented Jul 13, 2023

Copy link
Copy Markdown

Codecov Report

Merging #1080 (92f6efa) into main (a60d0ad) will decrease coverage by 0.25%.
The diff coverage is 68.75%.

❗ Current head 92f6efa differs from pull request most recent head c06fd83. Consider uploading reports for the commit c06fd83 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #1080      +/-   ##
============================================
- Coverage     81.65%   81.41%   -0.25%     
+ Complexity     2195     2189       -6     
============================================
  Files           280      280              
  Lines          6254     6243      -11     
============================================
- Hits           5107     5083      -24     
- Misses         1147     1160      +13     
Flag Coverage Δ
7.4 80.07% <68.75%> (-0.25%) ⬇️
8.0 81.35% <68.75%> (-0.25%) ⬇️
8.1 81.48% <68.75%> (-0.25%) ⬇️
8.2 81.48% <68.75%> (-0.25%) ⬇️

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

Impacted Files Coverage Δ
src/API/Baggage/Entry.php 0.00% <ø> (ø)
...ommon/Adapter/HttpDiscovery/DependencyResolver.php 91.66% <0.00%> (-8.34%) ⬇️
...DK/Common/Export/Stream/StreamTransportFactory.php 90.00% <ø> (ø)
src/SDK/Logs/SimplePsrFileLogger.php 100.00% <ø> (ø)
src/SDK/Resource/ResourceInfoFactory.php 54.54% <0.00%> (-16.61%) ⬇️
...rc/API/Trace/Propagation/TraceContextValidator.php 100.00% <100.00%> (ø)
src/API/Trace/SpanContextValidator.php 100.00% <100.00%> (ø)
src/Contrib/Otlp/HttpEndpointResolver.php 100.00% <100.00%> (ø)
src/Contrib/Zipkin/SpanConverter.php 96.29% <100.00%> (ø)
...n/Adapter/HttpDiscovery/HttpPlugClientResolver.php 100.00% <100.00%> (ø)
... and 2 more

... and 1 file 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 a60d0ad...c06fd83. Read the comment docs.

@brettmc
brettmc merged commit 29cff8b into open-telemetry:main Jul 19, 2023
@brettmc
brettmc deleted the static-analysis-fixes branch January 29, 2025 21:49
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