-
Notifications
You must be signed in to change notification settings - Fork 64
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
Conversation
This was triggered by: #78 (comment) |
48f05d4
to
eb4b54a
Compare
Sounds good. I'm OK with it as long as @jacobheun is OK with it :) |
There was a problem hiding this 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. 👍
@daviddias Tech lead, please approve :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing to update https://github.com/multiformats/js-multiaddr/blob/master/package.json#L5
Meanwhile, already gave @jacobheun publish perms |
eb4b54a
to
1a6f1ae
Compare
Good catch @daviddias, it's fixed now. |
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.