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.

Sign up for GitHub

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

Jump to bottom

deps: update proto-loader #4192

Merged
merged 3 commits into from
Oct 7, 2023
Merged

deps: update proto-loader #4192

merged 3 commits into from
Oct 7, 2023

Conversation

dyladan
Copy link
Member

@dyladan dyladan commented Oct 6, 2023

Update @grpc/proto-loader to latest

@dyladan dyladan requested a review from a team October 6, 2023 14:29
@codecov
Copy link

codecov bot commented Oct 6, 2023 •
edited
Loading

Codecov Report

Merging #4192 (7a7946b) into main (5fd3737) will increase coverage by 0.03%.
The diff coverage is n/a.

❗ Current head 7a7946b differs from pull request most recent head 4d9d2c8. Consider uploading reports for the commit 4d9d2c8 to get more accurate results

@@            Coverage Diff             @@
##             main    #4192      +/-   ##
==========================================
+ Coverage   92.22%   92.26%   +0.03%     
==========================================
  Files         330      331       +1     
  Lines        9446     9473      +27     
  Branches     1993     1999       +6     
==========================================
+ Hits         8712     8740      +28     
+ Misses        734      733       -1

see 2 files with indirect coverage changes

@dyladan dyladan added the dependencies Pull requests that update a dependency file label Oct 6, 2023
@legendecas legendecas merged commit c320c98 into open-telemetry:main Oct 7, 2023
16 checks passed
@dyladan dyladan deleted the update-proto-loader branch October 10, 2023 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pichlermarc pichlermarc pichlermarc approved these changes

@legendecas legendecas legendecas approved these changes

Assignees
No one assigned
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dyladan @legendecas @pichlermarc