conjoiners/conjoiners-node.js — pull request #1 by bripkens (2013-03-31 16:33:51+00:00)
Title: Project descriptor, build instructions, code fixes and more

Hi Pavlo,

good a few things for you:

- removed usage of require(...) for configuration file loading and exchanged it for fs.readFile(...) (require(...) is blocking). Added a callback for those interested in a finished conjoiner initialisation.
- tests can now be executed in any order and in one process. This was necessary for the following change:
- the project is now using Grunt for test execution and static source code checks. The README was extended with usage instructions. Based on this:
- fixed a few code issues, most importantly an implicit global looping variable :zap:

There are still a few open issues, e.g. the usage of __defineGetter__ and __defineSetter__ (both deprecated). I would propose to swap them for [defineProperty()](https://developer.mozilla.org/en-US/docs/JavaScript/Reference/Global_Objects/Object/defineProperty). I will open a separate issue for this.

Really like the idea of conjoiners!

Cheers

Ben