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

Remove anuraaga from transformprocessor reviewers #12219

Merged
merged 2 commits into from
Jul 13, 2022
Merged

Remove anuraaga from transformprocessor reviewers #12219

merged 2 commits into from
Jul 13, 2022

Conversation

anuraaga
Copy link
Contributor

@anuraaga anuraaga commented Jul 11, 2022 •
edited
Loading

Since not in org

@anuraaga anuraaga requested a review from a team as a code owner July 11, 2022 15:14
@anuraaga anuraaga requested a review from djaglowski July 11, 2022 15:14
@jmacd
Copy link
Contributor

jmacd commented Jul 11, 2022

@anuraaga I believe you are making an ad-hominem attack. If you would like to clarify your position in an issue at https://github.com/open-telemetry/community/issues, that might help us understand what you are trying to say.

@anuraaga
Copy link
Contributor Author

@jmacd You're right - removed the text

@TylerHelmuth
Copy link
Member

@anuraaga I added you as a code owner for the telemetryquerylanguage package. Are you stepping down from that as well?

@anuraaga
Copy link
Contributor Author

@TylerHelmuth Yes thanks for reminding. Got a high fever today so will update after it dies down

@codeboten
Copy link
Contributor

@TylerHelmuth Yes thanks for reminding. Got a high fever today so will update after it dies down

Updated it, thanks for all your contributions.

I hope your fever goes away soon!

@codeboten codeboten added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Jul 12, 2022
TylerHelmuth
TylerHelmuth approved these changes Jul 12, 2022
View reviewed changes
Copy link
Member

@TylerHelmuth TylerHelmuth left a comment

Choose a reason for hiding this comment

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

Thanks for all your contributions!

@codeboten codeboten merged commit e15878c into open-telemetry:main Jul 13, 2022
atoulme pushed a commit to atoulme/opentelemetry-collector-contrib that referenced this pull request Jul 16, 2022
@anuraaga @atoulme
Remove anuraaga from transformprocessor reviewers (open-telemetry#12219)
6a8aadd 
* Remove anuraaga from transformprocessor reviewers

* Update CODEOWNERS

Co-authored-by: Alex Boten <[email protected]>
ag-ramachandran referenced this pull request in ag-ramachandran/opentelemetry-collector-contrib Sep 15, 2022
@anuraaga @ag-ramachandran
Remove anuraaga from transformprocessor reviewers (#12219)
3d0717c 
* Remove anuraaga from transformprocessor reviewers

* Update CODEOWNERS

Co-authored-by: Alex Boten <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@codeboten codeboten codeboten approved these changes

@TylerHelmuth TylerHelmuth TylerHelmuth approved these changes

@djaglowski djaglowski Awaiting requested review from djaglowski djaglowski is a code owner automatically assigned from open-telemetry/collector-contrib-approvers

Assignees

@mx-psi mx-psi

Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@anuraaga @jmacd @TylerHelmuth @codeboten @mx-psi