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: Make @jacobheun the lead maintainer #81

Merged
merged 1 commit into from
Jan 28, 2019

Conversation

vmx
Copy link
Member

@vmx vmx commented Jan 25, 2019

I think it would make sense to hand the lead maintainership over to someone from the libp2p team, I propose @jacobheun.

What do you think @victorb?

Once approved my @victorb (current lead maintainer) and @daviddias (tech lead) I'll add jacob as maintainer to npm and will remove myself.

@vmx vmx requested review from victorb and daviddias January 25, 2019 13:33
@ghost ghost assigned vmx Jan 25, 2019
@ghost ghost added the status/in-progress In progress label Jan 25, 2019
@vmx
Copy link
Member Author

vmx commented Jan 25, 2019

This was triggered by: #78 (comment)

@vmx vmx force-pushed the change-lead-maintainer branch from 48f05d4 to eb4b54a Compare January 28, 2019 11:09
@victorb
Copy link
Member

victorb commented Jan 28, 2019

Sounds good. I'm OK with it as long as @jacobheun is OK with it :)

Copy link
Contributor

@jacobheun jacobheun left a comment

Choose a reason for hiding this comment

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

It's ok with me. 👍

@vmx
Copy link
Member Author

vmx commented Jan 28, 2019

@daviddias Tech lead, please approve :)

Copy link
Member

@daviddias daviddias left a comment

Choose a reason for hiding this comment

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

@daviddias
Copy link
Member

Meanwhile, already gave @jacobheun publish perms

@vmx vmx force-pushed the change-lead-maintainer branch from eb4b54a to 1a6f1ae Compare January 28, 2019 14:33
@vmx
Copy link
Member Author

vmx commented Jan 28, 2019

Good catch @daviddias, it's fixed now.

@daviddias daviddias merged commit 89de270 into master Jan 28, 2019
@daviddias daviddias deleted the change-lead-maintainer branch January 28, 2019 14:46
@ghost ghost removed the status/in-progress In progress label Jan 28, 2019
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.

4 participants