refactor npm packages - Githubissues

klaemo commented 9 years ago

hey, currently the project's structure isn't very best-practicey for the node world ;-)

dependencies/devDependencies should be specified in package.json
readme is missing from npm package
shrinkwrap is not strictly necessary
node_modules/tuio should be its own npm package, tuio-server for example

I can provide PRs for all of the above except the last one. :)

Are you open for these changes?

klaemo commented 9 years ago

also

socket.io can be updated to the latest version
libs should be depended on instead of manually exist in the repo

fe9lix commented 9 years ago

Hey,

sorry for the (very late) reply! The project's state is pretty much that of 2012, so I guess there are various areas for improvements. And yes, I'm open to changes - pull requests are fine!

nomve commented 8 years ago

Hi,

I've sent you a PR related to the dependencies/dev dependencies issue. I've left the shrinkwrap and socket.io issues alone, since other libs could also use an update. You can now install everything with npm install instead of manually installing tuio. The tuio or actually tuio server part really should be in its own repository. I als left libs alone since they are used in examples and tests. I could link the scripts in the node_modules directory, but then the tests wouldn't run without installing dependencies first. Not sure you want this. In general, the change is mostly cosmetic.

It would be best if the examples would use the dependencies bundled with say browserify and if the tests would run in PhantomJS without manually linking the scripts in index.html. I will probably do the second since I will try to implement some features of TUIO 2.0, but this is for some other time :).

fe9lix commented 8 years ago

Thx, merged!

fe9lix / Tuio.js

refactor npm packages #5