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

[chore] remove pjanotti from codeowners #23247

Merged
merged 1 commit into from
Jun 9, 2023
Merged

[chore] remove pjanotti from codeowners #23247

merged 1 commit into from
Jun 9, 2023

Conversation

atoulme
Copy link
Contributor

@atoulme atoulme commented Jun 9, 2023

This PR removes @pjanotti from multiple components. Please note that as result, the carbonexporter, carbonreceiver and wavefrontreceiver will no longer have active codeowners and may be therefore be considered unmaintained.

Please see issue #23246 for discussion.

@atoulme atoulme requested review from a team and fatsheep9146 June 9, 2023 03:11
@dmitryax
Copy link
Member

dmitryax commented Jun 9, 2023

Waiting for review from @pjanotti

pjanotti
pjanotti approved these changes Jun 9, 2023
View reviewed changes
Copy link
Contributor

@pjanotti pjanotti left a comment

Choose a reason for hiding this comment

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

Thanks @atoulme - I haven't been able to keep following the collector changes so it doesn't make sense to be still listed as a code owner.

@dmitryax dmitryax merged commit fbf6bbe into open-telemetry:main Jun 9, 2023
@github-actions github-actions bot added this to the next release milestone Jun 9, 2023
Caleb-Hurshman pushed a commit to observIQ/opentelemetry-collector-contrib that referenced this pull request Jul 6, 2023
@atoulme @Caleb-Hurshman
[chore] remove pjanotti from codeowners (open-telemetry#23247)
f507cef
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@crobert-1 crobert-1 crobert-1 approved these changes

@dmitryax dmitryax dmitryax approved these changes

@pjanotti pjanotti pjanotti approved these changes

@fatsheep9146 fatsheep9146 Awaiting requested review from fatsheep9146

Assignees

@jpkrohling jpkrohling

Labels
None yet
Projects
None yet
Milestone
v0.80.0
Development

Successfully merging this pull request may close these issues.
5 participants
@atoulme @dmitryax @pjanotti @crobert-1 @jpkrohling