Unable to install via npm due to preinstall requiring root access

OpenF2 / F2

Redefining web integration for the financial services community

Apache License 2.0

129 stars 62 forks source link

Closed kornelski closed 8 years ago

kornelski commented 9 years ago

package.json contains:

"scripts": {
    "preinstall": "npm install grunt-cli markitdown -g"
},

Global installation requires root privileges, and that's quite problematic for us, e.g. our CI servers don't allow it.

Could you move these packages to (dev)dependencies and use npm bin (./node_modules/.bin) instead?

markhealey commented 9 years ago

@pornel that was missed in our just-released v1.4.0. We replaced markitdown as part of #142. We'll get that preinstall cleaned up...

markhealey commented 9 years ago

@brianbaker please review and merge? Thanks.

qrider commented 8 years ago

preinstall has been removed from package.json in 1.4.1-wip