monterail / guidelines

[DEPRECATED] We are Ruby on Rails experts from Poland. Think hussars. Solid & winged. These are our guidelines.
71 stars 17 forks source link

Use Node.js instead of therubyracer for execjs runtime #175

Closed sheerun closed 10 years ago

sheerun commented 11 years ago

My reasons:

  1. Therubyracer is unstable between platforms, platform versions, ruby interpreters and even ruby versons. I personally couldn't install it on non-ree 1.8.7 and Windows (railsgirls), and had issues installing it on other combinations. Pretty much everytime there's native component compilation issue, it is therubyracer. I never had issues installing Node.js.
  2. therubyracer is officially discouraged by heroku as it uses lot of memory. One the other hand node.js is very lightweight.
  3. using therubyracer on server is just like installing node that can be used only by ruby apps (libv8)

btw. Node.js is necessary only on Windows and Ubuntu development, on Mac dev execjs is using JavaScriptCore.

jandudulski commented 11 years ago

makes sense :+1:

jandudulski commented 11 years ago

PR? @sheerun