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

chore: adding johnbley to js approvers #1845

Merged
merged 1 commit into from
Jan 20, 2021

Conversation

obecny
Copy link
Member

@obecny obecny commented Jan 19, 2021

I wanted to propose @johnbley as js approver. He is contributing to this project with web related stuff and I believe it would be great to have him as an approver too.

https://github.com/open-telemetry/opentelemetry-js-contrib/commits?author=johnbley
https://github.com/open-telemetry/opentelemetry-js/commits?author=johnbley

@codecov
Copy link

codecov bot commented Jan 19, 2021

Codecov Report

Merging #1845 (63e4d16) into master (1a3f3f8) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1845      +/-   ##
==========================================
- Coverage   92.62%   92.60%   -0.02%     
==========================================
  Files         174      174              
  Lines        6045     6045              
  Branches     1286     1286              
==========================================
- Hits         5599     5598       -1     
- Misses        446      447       +1     
Impacted Files Coverage Δ
...emetry-core/src/platform/node/RandomIdGenerator.ts 87.50% <0.00%> (-6.25%) ⬇️

Copy link
Member

@dyladan dyladan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Welcome to the team John

@johnbley
Copy link
Member

Welcome to the team John

Thanks!

@dyladan dyladan merged commit 08d5229 into open-telemetry:master Jan 20, 2021
@obecny obecny deleted the johnbley branch February 2, 2021 17:23
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.

6 participants