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

lint, tests, browser tests, readme update #4

Merged
merged 1 commit into from
Nov 6, 2015

Conversation

daviddias
Copy link
Member

bump minor :)

jbenet added a commit that referenced this pull request Nov 6, 2015
lint, tests, browser tests, readme update
@jbenet jbenet merged commit f2303c1 into multiformats:master Nov 6, 2015
@jbenet
Copy link
Member

jbenet commented Nov 6, 2015

@diasdavid thanks! published (btw you have owner on this)

@daviddias daviddias deleted the update branch November 6, 2015 10:33
@daviddias
Copy link
Member Author

Thank you :) I can't merge though #5

@daviddias
Copy link
Member Author

btw, protip: use npm-release like npm release patch, so that it creates a release with a tag and so on. @RichardLitt, you might want to add this to the JS boilarplate for our modules :)

@daviddias
Copy link
Member Author

@jbenet just noticed that I also don't have npm publish priv (was in a middle of a release and failed)

https://www.npmjs.com/package/multiaddr

@jbenet
Copy link
Member

jbenet commented Nov 6, 2015

try now?

On Fri, Nov 6, 2015 at 3:11 AM, David Dias notifications@github.com wrote:

@jbenet https://github.com/jbenet just noticed that I also don't have
npm publish priv (was in a middle of a release and failed)

https://www.npmjs.com/package/multiaddr


Reply to this email directly or view it on GitHub
#4 (comment).

@daviddias
Copy link
Member Author

sweet, done :)

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.

2 participants