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][receiver/cloudfoundry] Remove @pellared from CODEOWNERS #33071

Merged
merged 6 commits into from
May 15, 2024

Conversation

pellared
Copy link
Member

@pellared pellared commented May 15, 2024

The truth is that I am not currently a code owner of this component.

@pellared pellared requested review from a team and codeboten May 15, 2024 11:36
@andrzej-stencel
Copy link
Member

Sorry for the trouble, but the CODEOWNERS file in this repo is autogenerated from the components' metadata.yaml files.

You need to update the Cloud Foundry receiver's metadata.yaml file too (you can move yourself to emeritus code owners if you want).

@andrzej-stencel andrzej-stencel changed the title cloudfoundryreceiver: Remove @pellared from CODEOWNERS [chore] [receiver/cloudfoundry] Remove @pellared from CODEOWNERS May 15, 2024
@andrzej-stencel andrzej-stencel changed the title [chore] [receiver/cloudfoundry] Remove @pellared from CODEOWNERS [chore][receiver/cloudfoundry] Remove @pellared from CODEOWNERS May 15, 2024
@andrzej-stencel andrzej-stencel added the Skip Changelog PRs that do not require a CHANGELOG.md entry label May 15, 2024
@pellared

This comment was marked as outdated.

@pellared pellared marked this pull request as draft May 15, 2024 11:58
@pellared pellared marked this pull request as ready for review May 15, 2024 12:00
@pellared pellared marked this pull request as draft May 15, 2024 12:12
@pellared
Copy link
Member Author

Sorry for the trouble, but the CODEOWNERS file in this repo is autogenerated from the components' metadata.yaml files.

@andrzej-stencel, this is good 👍 I think I updated the PR correctly. I make the PR as "Ready for review" after the GH workflows pass.

@pellared pellared marked this pull request as ready for review May 15, 2024 13:28
Copy link
Member

@crobert-1 crobert-1 left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for your help here @pellared!

@crobert-1 crobert-1 added the ready to merge Code review completed; ready to merge by maintainers label May 15, 2024
@andrzej-stencel andrzej-stencel merged commit bfa2800 into open-telemetry:main May 15, 2024
169 checks passed
@github-actions github-actions bot added this to the next release milestone May 15, 2024
@pellared pellared deleted the patch-1 branch May 15, 2024 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Code review completed; ready to merge by maintainers receiver/cloudfoundry Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants