These are chat archives for jescalan/roots

8th
May 2016
Tom Kraak
@tkraak
May 08 2016 17:08

node and roots newbie here … I have a large node_modules directory in my project after doing roots new my-project … that’s normal for node apps, no?

thanks!

Ben Styles
@benadamstyles
May 08 2016 17:13
Yep totally normal!
If you're using git, general practise is not to check in node_modules, and instead to check in package.json and for new contributors to npm install when they clone your repo or pull changes. For that you need to remember to npm install <new-package> --save or --save-dev.
If you want exact package versions checked into git, use npm-shrinkwrap.json (google it!).
Tom Kraak
@tkraak
May 08 2016 17:16
got it, thanks!
Ben Styles
@benadamstyles
May 08 2016 17:16
np
Tom Kraak
@tkraak
May 08 2016 17:17
and I don’t have to worry about that roots will “move” node_modules into public, right?
Ben Styles
@benadamstyles
May 08 2016 17:33
I don't think so, but if it does you can just add 'node_modules' to app.ignores in app.coffee
But I think it ignores it by default
Tom Kraak
@tkraak
May 08 2016 17:34
I think so too, thx
Ben Styles
@benadamstyles
May 08 2016 17:34
Are you bundling your JS with webpack or browserify or any other build step?
Tom Kraak
@tkraak
May 08 2016 17:34
not quite there yet :)
Ben Styles
@benadamstyles
May 08 2016 17:35
:) well anyway I think it should work as you expect. node_modules is not in my app.ignores and it's still not copied over.
Tom Kraak
@tkraak
May 08 2016 17:35
roots watch won’t run
Ben Styles
@benadamstyles
May 08 2016 17:35
any errors?
Tom Kraak
@tkraak
May 08 2016 17:39
npm WARN deprecated graceful-fs@3.0.8: graceful-fs v3.0.0 and before will fail on node releases >= v7.0.
Ben Styles
@benadamstyles
May 08 2016 17:39
that's just a warning, won't cause anything to fail
nothing else?
Windows/Linux/Mac?
Tom Kraak
@tkraak
May 08 2016 17:40
mac
(node:81163) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more re
cent version.

Error: TypeError: /Users/tkraak/Learn/roots/learn-roots/assets/css/master.styl:8:1
4| base()
5|
6| body
7| padding: 50px
8|
------^

Path must be a string. Received undefined

Ben Styles
@benadamstyles
May 08 2016 18:07
Looks like a stylus error, I'm afraid I've not used stylus before so I don't know what the syntax should look like, but it looks like you haven't indented line 7? I think that should be indented 2 spaces.
Tom Kraak
@tkraak
May 08 2016 18:43
I have not touched anything yet, that’s the very first roots watch
Ben Styles
@benadamstyles
May 08 2016 19:00
Oh. Maybe there's a bug in the initial template. Have you tried indenting that padding line? Curious to see if I was right...
Tom Kraak
@tkraak
May 08 2016 19:03
(node:76227) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.
that’s still just a warning?
Ben Styles
@benadamstyles
May 08 2016 20:17
What version of node are you using?
Tom Kraak
@tkraak
May 08 2016 20:18
6.1
I got npm issues too :(
npm ERR! Cannot find module 'read-package-json'
which means I can’t insall or upgrade anything
Ben Styles
@benadamstyles
May 08 2016 20:19
Have you just updated? When you use a new version of node using nvm, your global packages don't come with you. You're using a whole new version, with its own global packages location.
Tom Kraak
@tkraak
May 08 2016 20:19
I actually used n to upgrade to 6 last week
something must have gone wrong then
what’s a good way to start over “clean” with node and npm, etc?
is that the way to go?
back later, thank you!
seems like the way to go