Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Semver Minor Version bumpup (7.5.1 -> 7.5.2) #4076

Merged
merged 6 commits into from
Aug 23, 2023

Conversation

shashi3kiran3
Copy link
Contributor

…3996)

Which problem is this PR solving?

This PR resolved Semver vulnerability in 7.5.1 version

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Updated package details in package.json

Fixes # (issue)

GHSA-c2qf-rxjj-qqgw

Short description of the changes

Semver Minor Version bumpup (7.5.1 -> 7.5.2)

Type of change

Semver Minor Version bumpup (7.5.1 -> 7.5.2)

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
NA

  • Test A

Checklist:

  • Followed the style guidelines of this project
  • Unit tests have been added
  • Documentation has been updated

@shashi3kiran3 shashi3kiran3 requested a review from a team August 17, 2023 16:47
@dyladan dyladan added the dependencies Pull requests that update a dependency file label Aug 23, 2023
@codecov
Copy link

codecov bot commented Aug 23, 2023

Codecov Report

Merging #4076 (ca31be2) into main (92050b5) will increase coverage by 1.86%.
The diff coverage is n/a.

❗ Current head ca31be2 differs from pull request most recent head 8c5a198. Consider uploading reports for the commit 8c5a198 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4076      +/-   ##
==========================================
+ Coverage   90.52%   92.38%   +1.86%     
==========================================
  Files         159      321     +162     
  Lines        3756     9273    +5517     
  Branches      835     1970    +1135     
==========================================
+ Hits         3400     8567    +5167     
- Misses        356      706     +350     

see 167 files with indirect coverage changes

@dyladan dyladan added the Merge:LGTM This PR is ready to be merged by a Maintainer (has enough valid approvals, successful build, etc.) label Aug 23, 2023
@dyladan dyladan merged commit 863c4d4 into open-telemetry:main Aug 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file Merge:LGTM This PR is ready to be merged by a Maintainer (has enough valid approvals, successful build, etc.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants