Skip to content

Fix PSR-3 module description, standardize other PSR descriptions#240

Merged
brettmc merged 1 commit into
open-telemetry:mainfrom
agoallikmaa:psr-descriptions-fix
Feb 28, 2024
Merged

Fix PSR-3 module description, standardize other PSR descriptions#240
brettmc merged 1 commit into
open-telemetry:mainfrom
agoallikmaa:psr-descriptions-fix

Conversation

@agoallikmaa

Copy link
Copy Markdown
Contributor

PSR-3 module description was incorrectly describing it as HTTP client rather than logging instrumentation. Changed all PSR module descriptions to use the format PSR-<number> (<official_name>) based on the name used on PHP-FIG site.

@agoallikmaa
agoallikmaa requested a review from a team February 26, 2024 04:34
@codecov

codecov Bot commented Feb 26, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.10%. Comparing base (5ef0ba0) to head (0d78090).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #240      +/-   ##
============================================
- Coverage     85.55%   85.10%   -0.45%     
+ Complexity      966      920      -46     
============================================
  Files            88       83       -5     
  Lines          3917     3612     -305     
============================================
- Hits           3351     3074     -277     
+ Misses          566      538      -28     
Flag Coverage Δ
7.4 85.72% <ø> (ø)
8.0 84.00% <ø> (-0.70%) ⬇️
8.1 84.14% <ø> (-0.67%) ⬇️
8.2 86.49% <ø> (+1.02%) ⬆️

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

see 5 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 5ef0ba0...0d78090. Read the comment docs.

@brettmc
brettmc merged commit 7edb8be into open-telemetry:main Feb 28, 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.

4 participants