-
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
Lead maintainer #66
Lead maintainer #66
Conversation
package.json
Outdated
@@ -2,6 +2,7 @@ | |||
"name": "multiaddr", | |||
"version": "5.0.0", | |||
"description": "multiaddr implementation (binary + string representation of network addresses)", | |||
"lead-maintainer": "Victor Bjelkholm <[email protected]>", |
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.
shouldn't this be leadMaintainer
to follow the rest of the casing of keys in package.json?
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.
Good point. Now I've to change it all xD
@@ -17,10 +17,6 @@ | |||
"coverage-publish": "aegir coverage publish", | |||
"docs": "aegir docs" | |||
}, | |||
"pre-commit": [ |
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.
This should not be removed, but changed to pre-push
. I'd also like to see a better commit message :)
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.
@victorbjelkholm can you handle the changes?
For ipfs/community#301