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

Move @bryan-aguilar to emeritus #34186

Closed
atoulme opened this issue Jul 21, 2024 · 4 comments · Fixed by #34523
Closed

Move @bryan-aguilar to emeritus #34186

atoulme opened this issue Jul 21, 2024 · 4 comments · Fixed by #34523

Comments

@atoulme
Copy link
Contributor

atoulme commented Jul 21, 2024 •
edited
Loading 

          Hi @dmitryax, sorry to ping you for this PR that wasn't assigned to you. @bryan-aguilar isn't active in OpenTelemetry, I suppose we may need to move him to `emeritus` status. Could you help again to approve/merge this PR in his place?

Originally posted by @jj22ee in #33798 (comment)

@bryan-aguilar please voice if you believe you should not move to emeritus status.
@github-actions github-actions bot mentioned this issue Jul 23, 2024
Closed
@mx-psi mx-psi mentioned this issue Aug 7, 2024
Merged
@mx-psi
Copy link
Member

mx-psi commented Aug 7, 2024

I removed @bryan-aguilar from the review auto-assignment so that PRs get reviews until we get a reply/move to emeritus

codeboten pushed a commit that referenced this issue Aug 7, 2024
@mx-psi @songy23
[chore][CONTRIBUTING.md] Outline rules for moving contributors to eme…
32746f2 
…ritus (#34463)

Outlines rules for moving a contributor to emeritus from an existing
role.

**Link to tracking Issue:** Relates to #34186

---------

Co-authored-by: Yang Song <[email protected]>
@crobert-1
Copy link
Member

I removed @bryan-aguilar from the review auto-assignment so that PRs get reviews until we get a reply/move to emeritus

FYI: It looks like this may not have been completed successfully, a PR opened today was assigned to @bryan-aguilar.

@codeboten codeboten mentioned this issue Aug 8, 2024
Merged
@mx-psi
Copy link
Member

mx-psi commented Aug 8, 2024

I changed the auto-assignment for reviewers, but not for assignees (sorry, it's confusing). The assignees are handled via a file somewhere, we can also make this change

@crobert-1
Copy link
Member

I changed the auto-assignment for reviewers, but not for assignees (sorry, it's confusing). The assignees are handled via a file somewhere, we can also make this change

Ah, my bad, thanks for clarifying! Looks like the assignment should be resolved by #34523 👍

f7o pushed a commit to f7o/opentelemetry-collector-contrib that referenced this issue Sep 12, 2024
@mx-psi @songy23 @f7o
[chore][CONTRIBUTING.md] Outline rules for moving contributors to eme…
f75df18 
…ritus (open-telemetry#34463)

Outlines rules for moving a contributor to emeritus from an existing
role.

**Link to tracking Issue:** Relates to open-telemetry#34186

---------

Co-authored-by: Yang Song <[email protected]>
f7o pushed a commit to f7o/opentelemetry-collector-contrib that referenced this issue Sep 12, 2024
@codeboten @f7o
move bryan-aguilar to emeritus (open-telemetry#34523)
268f431 
Thanks for all your contributions @bryan-aguilar

Closes open-telemetry#34186

---------

Signed-off-by: Alex Boten <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

move bryan-aguilar to emeritus codeboten/opentelemetry-collector-contrib
3 participants
@atoulme @mx-psi @crobert-1