Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: upgrading to new aegir #17

Merged
merged 3 commits into from
Dec 13, 2017
Merged

feat: upgrading to new aegir #17

merged 3 commits into from
Dec 13, 2017

Conversation

dryajov
Copy link
Contributor

@dryajov dryajov commented Dec 13, 2017

No description provided.

@daviddias
Copy link
Owner

@dryajov there was one timeout in CI, otherwise LGTM :) thanks!

@@ -2,8 +2,7 @@
"name": "webcrypto",
"version": "0.1.0",
"description": "Use the Node.js crypto module API without having to worry if it is being run in browser or Node.js",
"main": "lib/index.js",
Copy link
Owner

Choose a reason for hiding this comment

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

Really old module xD

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah, couldn't figure out why it was failing when linked in js-multihashing :D

@daviddias daviddias merged commit ab52405 into daviddias:master Dec 13, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants