-
Notifications
You must be signed in to change notification settings - Fork 63
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
haxelib release #40
Comments
Well, there's two things.
I haven't looked through the source extensively, but it might be worthwhile detecting other places that lack type safety. We can still get rid of it later on, if it turns out too clunky. The other way round is not possible though. |
I've been using it in a "real world" project.. with Http related features 2015-07-21 9:26 GMT-03:00 Juraj Kirchheim [email protected]:
[image: Imagem inline 1] |
Yeah we should review TODOs but wrt type safety I'm not sure. I remember we wanted to have good balance between typesafe and being able to copy-paste js code with minimal changes. |
Same experience as Eduardo and I would agree that we can release with a 0.x Il giorno mar 21 lug 2015 alle ore 08:37 Dan Korostelev <
|
Also me and other member have our own libs for nodejs libs bindings! |
@nadako I understand what you are saying. There is of course no reason to make copy pasting particularly hard. That being said, the differences between Haxe and JS are there. Node's module system certainly doesn't make it easier (although I will say that it's module philosophy also decreases the need compared to frontend JS). My broader stance is that we can only seek that balance walking one way through the spectrum, so I think that it might be wiser to get all the room to maneuver we can get (with reasonable effort) and then let user feedback govern how far we go back. But I trust you on making that decision. One thing I will add though is that in future versions we should be able to use |
So I was cleaning up my repo today, |
With regard to version, I thought we agreed on syncing major/minor versions with node.js API version and using patch versions for hxnodejs releases (see #26 (comment)) |
I'm not too sure about syncing with node.js API versions ? |
I have just switched my only NodeJS project (CastleDB) from nodejs-std to hxnodejs, it works very well. I have commit js.node.webkit package and started sys package + Sys class implementation. If anyone (@dionjwa ?) wants to contribute to complete these that would be nice. Also, @nadako, before releasing we should decide on the library name on haxelib. If I remember correclty, @dionjwa proposed that we take over http://lib.haxe.org/p/nodejs/ instead of adding one more nodejs library. That seems like a good idea. We could also ask @eduardo-costa if we wants to contribute to hxnodejs instead of his own nodehx lib. Let's all join efforts into one single nodejs lib ! |
We that would be great!
|
Yes, please use the 'nodejs' haxelib, that makes the most sense I think. Let me know who needs access and I'll add you as a contributor. When that's in, I'll definitely help with filling in the stdlib. |
Wouldn't delay even more we closing and releasing hxnodejs?
|
Guys! I proposed days ago we merge all our work there. We should create each packages matching the
|
ok guys |
@clemos @dionjwa Feel free to transfer the nodejs libs you have made externs and delete any of mine you think isn't complete enough versus your versions! @ncannasse The |
I like the distinction @clemos makes between e.g. 'js.node.Fs' and 'js.npm.Express'. There's a clear distinction at the package level between packages that are part of the Node.js distribution, and npm packages. |
I'm ok with that too..
|
@ncannasse I thought we've all agreed on the idea that hxnodejs (or whatever name we choose) should only contain core node.js api, so I'm not sure whether placing webkit externs there is a good decision, maybe it should be contained in our hxnodelibs repo? (also, those node-webkit externs are incomplete and undocumented unlike core node.js externs). As for the name, to be honest anything is fine with me, but IIRC we named it Anyway, I love the fact that hxnodejs is getting some attention and sorry I'm a little distracted right now (as I'm on vacation). |
@nadako sorry for webkit, but I think it will be necessary anyway to make some specific changes in hxnodejs wrt node webkit implementation. The reason is that in a pure NodeJS context you might want to have haxe.io.Bytes.BytesData be a NodeBuffer, so you don't have to copy from/to UInt8Array. But I'll leave the decision to you and other maintainers, if you want to keep it there or have it into hxnodelibs. WRT the name, choose also what is the best for you, I just want to make sure that the most complete/maintained library get first results when searching on haxelib :) |
I think that NodeWebkit falls into the |
Anything new regarding the release ? |
I don't know. I was working on updating externs to node.js 4.0.0 and it's not done yet, but I have a feeling that we'll never catch up to it, so maybe we should just make a "alpha" release and then update it with fixes/changes. I'm still not sure what to do with the #if js
#if nodejs // or some other flag
...nodejs impl...
#else
...browser impl or error...
#end
#end |
Yeah I still think it would be cool to make a release of the current pre-node 4.0.0 version, so we can all link to it. |
On Thu, Sep 17, 2015 at 2:24 PM, Dan Korostelev [email protected]
Best, |
@back2dos ah, great news ! problem solved :) hopefully NodeWebkit will upgrade to node 4.0 soon as well |
@ncannasse Have you tried Electron btw? How does it compare to NW? |
@nadako didn't knew about it, might be better I'll give it a try one day |
Here is a comparison: https://github.com/atom/electron/blob/master/docs/development/atom-shell-vs-node-webkit.md |
Just remembering.. |
Up ? |
I really wish to see a release. Currently using Haxe/Nodejs and js-kit lib for a professional project. |
I am currently using it a lot and good with a release. |
#42 is definitely a show stopper for a release, we can either release without haxe.io/sys packages, or find a way to fix this. |
I prefer a release first, even just 0.0.1, so that other lib can start using it |
Okay, it's on haxelib now. |
YAY! |
👍 |
gg wp :) |
We should make a release of hxnodejs at some point, but I'm still reluctant to do that because I haven't receive much feedback on general structure and API design of the library which will be harder to change after an official release.
So please guys, speak up now and feel free to mention other people who are already using hxnodejs.
@fponticelli @clemos @ncannasse @ousado @frabbit @eduardo-costa @tong @back2dos
The text was updated successfully, but these errors were encountered: