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

Remove iredelmeier as a JS approver #218

Merged

Conversation

iredelmeier
Copy link
Member

[Closes #217]

@bogdandrutu
Copy link
Member

Can you please elaborate a bit on the reasoning? Is this just because you no longer want to be involved in that project? Is because you don't feel you have enough time? Or because you cannot work with people from that SIG?

Also something like "personal decision" is fine (if you don't want to share details), but I try to understand if we can do something to help.

Copy link
Member

@SergeyKanzhelev SergeyKanzhelev left a comment

Choose a reason for hiding this comment

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

+1 to what @bogdandrutu asked. Approved anyway as it's self-removal request

@iredelmeier
Copy link
Member Author

@bogdandrutu @SergeyKanzhelev I got grandfathered in (or something like that) when really I'm not actively contributing to the JS SDK. Definitely nothing to do with personal problems or other problems with the people!

@bhs
Copy link
Contributor

bhs commented Oct 14, 2019

I agree that it would be great to learn anything @iredelmeier would like to share about how this SIG, this repo, and/or OTel in general can improve, but in the meantime I don't think there should be any requirement for explanation before merging a self-request to leave a SIG; so I will merge this.

(In parallel, @iredelmeier has provided her rationale, so I'm feeling even further emboldened ;)

@bhs bhs merged commit 1729e60 into open-telemetry:master Oct 14, 2019
@bogdandrutu
Copy link
Member

@bhs sorry if it was miss understand that it was a requirement, I was asking to learn how we can improve the experience for developers if there was a problem.

Thanks @iredelmeier for the clarification.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

REQUEST: Remove me as a JS approver
4 participants