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

Remove @bg451 from @open-telemetry/js-approvers #1264

Merged

Conversation

mayurkale22
Copy link
Member

  • This is based on offline discussion with @bg451.

  • @bg451 was in the original team to kick off this project

  • Many thanks to @bg451 for your contributions and valuable reviews on the many PRs especially OpenTracing-shim and Metrics-API work.

/cc @dyladan @obecny

@codecov
Copy link

codecov bot commented Jul 1, 2020

Codecov Report

Merging #1264 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1264   +/-   ##
=======================================
  Coverage   92.92%   92.92%           
=======================================
  Files         131      131           
  Lines        3674     3674           
  Branches      742      742           
=======================================
  Hits         3414     3414           
  Misses        260      260           

@mayurkale22 mayurkale22 merged commit ac7dc6d into open-telemetry:master Jul 1, 2020
@mayurkale22 mayurkale22 deleted the update_approvers_list_1 branch July 1, 2020 15:51
dyladan pushed a commit to dyladan/opentelemetry-js that referenced this pull request Sep 9, 2022
dyladan pushed a commit to dyladan/opentelemetry-js that referenced this pull request Sep 9, 2022
pichlermarc pushed a commit to dynatrace-oss-contrib/opentelemetry-js that referenced this pull request Dec 15, 2023
martinkuba pushed a commit to martinkuba/opentelemetry-js that referenced this pull request Mar 13, 2024
martinkuba pushed a commit to martinkuba/opentelemetry-js that referenced this pull request Mar 16, 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