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

Update Maintainer for Otel Go. #660

Merged
merged 4 commits into from
Apr 24, 2020
Merged

Update Maintainer for Otel Go. #660

merged 4 commits into from
Apr 24, 2020

Conversation

rghetia
Copy link
Contributor

@rghetia rghetia commented Apr 23, 2020

  • Replace Rahul Patel with Tyler Yahn as a maintainer.

@rghetia
Update Maintainer for Otel Go.
43ef4c8 
- Replace Rahul Patel with Tyler Yahn as maintainer.
@rghetia
Copy link
Contributor Author

rghetia commented Apr 23, 2020

/cc @MrAlias @lizthegrey

@lizthegrey
Copy link
Member

Rahul, would you like to stay as an approver, or are you rotating off entirely?

@rghetia
Copy link
Contributor Author

rghetia commented Apr 23, 2020

Rahul, would you like to stay as an approver, or are you rotating off entirely?

I am rotating off entirely.

krnowak
krnowak approved these changes Apr 24, 2020
View reviewed changes
Copy link
Member

@krnowak krnowak left a comment

Choose a reason for hiding this comment

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

Who's going to be a point of contact for the vanity URLs?

lizthegrey
lizthegrey suggested changes Apr 24, 2020
View reviewed changes
Copy link
Member

@lizthegrey lizthegrey left a comment

Choose a reason for hiding this comment

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

Make sure to change CODEOWNERS too.

lizthegrey
lizthegrey reviewed Apr 24, 2020
View reviewed changes
CODEOWNERS Outdated Show resolved Hide resolved
@rghetia
Copy link
Contributor Author

rghetia commented Apr 24, 2020 •
edited
Loading

Who's going to be a point of contact for the vanity URLs?

I am still the POC until I provide my replacement.

lizthegrey
lizthegrey approved these changes Apr 24, 2020
View reviewed changes
Copy link
Member

@lizthegrey lizthegrey left a comment

Choose a reason for hiding this comment

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

Sorry to see you go, and best of luck with what comes next.

@rghetia rghetia merged commit 52b2534 into open-telemetry:master Apr 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@krnowak krnowak krnowak approved these changes

@lizthegrey lizthegrey lizthegrey approved these changes

@jmacd jmacd jmacd approved these changes

@MrAlias MrAlias MrAlias approved these changes

@paivagustavo paivagustavo Awaiting requested review from paivagustavo

@tedsuo tedsuo Awaiting requested review from tedsuo

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@rghetia @lizthegrey @krnowak @jmacd @MrAlias