Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

npm install not working in GitHub actions since version 2 #12

Closed
P4sca1 opened this issue Jun 8, 2020 · 2 comments · Fixed by #13
Closed

npm install not working in GitHub actions since version 2 #12

P4sca1 opened this issue Jun 8, 2020 · 2 comments · Fixed by #13

Comments

@P4sca1
Copy link

P4sca1 commented Jun 8, 2020

[4/4] Building fresh packages...
error /home/node/node_modules/zingchart-vue: Command failed.
Exit code: 243
Command: npm install -g conventional-changelog-cli
Arguments: 
Directory: /home/node/node_modules/zingchart-vue
Output:
npm WARN checkPermissions Missing write access to /usr/local/lib/node_modules
npm ERR! code EACCES
npm ERR! syscall access
npm ERR! path /usr/local/lib/node_modules
npm ERR! errno -13
npm ERR! Error: EACCES: permission denied, access '/usr/local/lib/node_modules'
npm ERR!  [Error: EACCES: permission denied, access '/usr/local/lib/node_modules'] ***
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   syscall: 'access',
npm ERR!   path: '/usr/local/lib/node_modules'
npm ERR! ***
npm ERR! 
npm ERR! The operation was rejected by your operating system.
npm ERR! It is likely you do not have the permissions to access this file as the current user
npm ERR! 
npm ERR! If you believe this might be a permissions issue, please double-check the
npm ERR! permissions of the file and its containing directories, or try running
npm ERR! the command again as root/Administrator.

Version 1 used to work fine.

@P4sca1
Copy link
Author

P4sca1 commented Jun 8, 2020

A package should not try to install another package globally upon installation.

@P4sca1
Copy link
Author

P4sca1 commented Jun 8, 2020

The preinstall script which installs conventional-changelog-cli globally should be removed. Instead it should be added to devDependencies.

damntrecky pushed a commit that referenced this issue Jun 8, 2020
Remove the preinstall npm script

#12
@damntrecky damntrecky linked a pull request Jun 8, 2020 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant