Skip to content

fix: npm warn install-scripts - #967

Closed
fastnloud wants to merge 1 commit into
dunglas:mainfrom
fastnloud:fix/npm-warn-install-scripts
Closed

fastnloud wants to merge 1 commit into
dunglas:mainfrom
fastnloud:fix/npm-warn-install-scripts

Conversation

@fastnloud

Copy link
Copy Markdown

This should fix the following warning:

npm warn install-scripts 1 package has install scripts not yet covered by allowScripts:
npm warn install-scripts   protobufjs@8.3.0 (postinstall: node scripts/postinstall)
npm warn install-scripts
npm warn install-scripts Run `npm install -g --allow-scripts=protobufjs` to allow these scripts once, or `npm config set allow-scripts=protobufjs --location=user` to allow them for all global installs.

@7-zete-7

Copy link
Copy Markdown
Contributor

Hi @fastnloud!

It looks like this issue has already been reported in protobufjs/protobuf.js#2296 and fixed in protobufjs/protobuf.js#2299. The postinstall script no longer exists since protobufjs v8.6.0. intellephense uses protobufjs v8.3.0, which had the same functionality of the the postinstall script. I'm guessing adding this option isn't necessary.

@fastnloud

Copy link
Copy Markdown
Author

Thanks @7-zete-7 for looking into it. I think you might be right and that it'll fix itself whenever intellephense decides to up this dependency.

Feel free to close this if no longer relevant.

@maxhelias

Copy link
Copy Markdown
Collaborator

Thank you both

@maxhelias maxhelias closed this Sep 24, 2026
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 this pull request may close these issues.

3 participants