-
Notifications
You must be signed in to change notification settings - Fork 63
feat(init): Add the possibility to pass options to init() #123
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
Conversation
@dignifiedquire can I get a CR for this? |
@haadcode can I get a test for that :) |
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.
Needs a test
6fd1d02
to
9f1a24c
Compare
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.
one small thing, other than that Lgtm, thank you
@@ -345,8 +371,9 @@ describe('ipfs-api version', function () { | |||
}) | |||
}) | |||
|
|||
// NOTE: if you change ../src/, the hash will need to be changed |
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 be if you change fixtures
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.
Fixed
7bc885e
to
2bb5dd7
Compare
Thank you @haadcode |
🙌🏼 |
This PR will change the local() api so that options object can be passed to it.
PRing this again (original #109).