-
Notifications
You must be signed in to change notification settings - Fork 72
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
QUESTION: Forking? #14
Comments
Hey Eric! nice work! It is definitely your decision whether you want to create a new package or contribute your changes back to this project. I went ahead and added you as collaborator if you want to merge your changes back. I already reviewed your code too and it looks good to me! |
I'll definitely merge it back to this repo here when I get some time. Thanks for adding me as collaborator and thanks for the initial code base. This is pretty amazing |
I'm glad to hear that. Looking forward to it.
|
Question on copyright: I thought about just removing those, but I'm not sure what the implication would be, if there are any. |
Yeah, I re-licensed all my open source projects to use MPL2. You can use either your IDE or https://github.com/c4milo/licentia to do that. |
I'm looking in to publishing the code on npmjs.org. |
@thejinx0r looking into it |
@thejinx0r done. |
Hi c4milo,
I liked the idea of using the fuse low-level api (especially with the asynchronous api) with nodeJS.
The code you had written was not working, and there are several issues suggesting a rewrite is needed.
I've made some minor (or maybe major?) modifications to it and it now works. If anyone is interested, have a look at https://github.com/thejinx0r/fusejs/tree/dev. The performance is great because fuse can run in multithreaded mode and the developer does not need to worry about that on the JS side!
I was thinking of releasing the changes back to the public, and I was wondering how should I proceed?
Should I create a new package and acknowledge your work, or create a pull request and merge the code in?
I haven't enabled all the functions yet, but it should be straightforward to get everything to work.
Eric
The text was updated successfully, but these errors were encountered: