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

Add dmitryax as an approver #895

Conversation

tigrannajaryan
Copy link
Member

Dmitrii has been actively working on the Collector and has
already contributed 23 commits, most of them quite substantial.

I vouch for him.

Dmitrii has been actively working on the Collector and has
already contributed 23 commits, most of them quite substantial.

I vouch for him.
@codecov-io
Copy link

Codecov Report

Merging #895 into master will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #895      +/-   ##
==========================================
+ Coverage   85.94%   85.96%   +0.01%     
==========================================
  Files         172      172              
  Lines       13136    13136              
==========================================
+ Hits        11290    11292       +2     
+ Misses       1419     1417       -2     
  Partials      427      427              
Impacted Files Coverage Δ
translator/internaldata/resource_to_oc.go 73.52% <0.00%> (ø)
service/service.go 66.33% <0.00%> (+0.99%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 676261f...ea732c9. Read the comment docs.

@bogdandrutu
Copy link
Member

Please leave this PR for 1 day at least open.

@bogdandrutu bogdandrutu merged commit 8cb080d into open-telemetry:master May 1, 2020
@tigrannajaryan tigrannajaryan deleted the feature/tigran/new-approver branch May 1, 2020 15:24
tigrannajaryan pushed a commit to tigrannajaryan/opentelemetry-collector-contrib that referenced this pull request May 1, 2020
dmitryax is already an approver on the Collector core:
open-telemetry/opentelemetry-collector#895

This ensures he has the approving permission in this repo too.
bogdandrutu pushed a commit to open-telemetry/opentelemetry-collector-contrib that referenced this pull request May 1, 2020
dmitryax is already an approver on the Collector core:
open-telemetry/opentelemetry-collector#895

This ensures he has the approving permission in this repo too.
wyTrivail pushed a commit to mxiamxia/opentelemetry-collector that referenced this pull request Jul 13, 2020
Dmitrii has been actively working on the Collector and has
already contributed 23 commits, most of them quite substantial.

I vouch for him.
wyTrivail pushed a commit to mxiamxia/opentelemetry-collector-contrib that referenced this pull request Jul 13, 2020
dmitryax is already an approver on the Collector core:
open-telemetry/opentelemetry-collector#895

This ensures he has the approving permission in this repo too.
Troels51 pushed a commit to Troels51/opentelemetry-collector that referenced this pull request Jul 5, 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.

5 participants