npm install results in git seeing local changes about syn

[matthewp]

I don't understand, what does prepublish have to do with install?

[mickmcgrath13]

idk. I run npm install, and a build runs if "prepublish" is there. Does scripts run on npm install?

[mickmcgrath13]

https://docs.npmjs.com/misc/scripts

prepublish: Run BEFORE the package is published. (Also run on local npm install without any arguments.)

[matthewp]

Oh, I think it's fine that it does the build. It's something we do in several projects. The reason we created that is to make sure the build happens before publish.

[mickmcgrath13]

lol @ npm/npm#3059

..apparently "prepublish" running on npm install is kind of a big deal, and that's just the way it works.

Also, npm install --ignore-scripts does the install the way I would expect, but I would think the general case for people should be clone, and npm install... I wouldn't expect people to remember the --ignore-scripts part, and without it, they'll get local file changes.

[matthewp]

Yeah, we should gitignore the dist/ folder in master and only publish it in a tag by force adding.

[mickmcgrath13]

#103

[mickmcgrath13]

closing in lieu of #106