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.

Sign up for GitHub

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

Jump to bottom

[DOCS] option foreground-scripts is not documented in npm install documentation #3820

Closed
1 task done
hasezoey opened this issue Oct 1, 2021 · 2 comments · Fixed by #4580
Closed
1 task done

[DOCS] option foreground-scripts is not documented in npm install documentation #3820

hasezoey opened this issue Oct 1, 2021 · 2 comments · Fixed by #4580
Labels
Bug thing that needs fixing Priority 2 secondary priority issue Release 7.x work is associated with a specific npm 7 release

Comments

@hasezoey
Copy link

hasezoey commented Oct 1, 2021

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

I had already mentioned this issue in #3347, which i though was fixed by #3360, but documentation at https://docs.npmjs.com/cli/v7/commands/npm-install still does not mention the option foreground-scripts

Expected Behavior

To have Documentation about foreground-scripts

Steps To Reproduce

open https://docs.npmjs.com/cli/v7/commands/npm-install and search for foreground-scripts

Environment

Online Documentation
@hasezoey hasezoey added Bug thing that needs fixing Needs Triage needs review for next steps Release 7.x work is associated with a specific npm 7 release labels Oct 1, 2021
@hasezoey hasezoey mentioned this issue Oct 1, 2021
Closed
@jiaqidegege
Copy link

It's mentioned here: https://docs.npmjs.com/cli/v7/using-npm/scripts

@hasezoey
Copy link
Author

hasezoey commented Oct 6, 2021

It's mentioned here: https://docs.npmjs.com/cli/v7/using-npm/scripts

yes it is mentioned there, but from what i know it is a option to npm install and so should be mentioned as a option there

PS: your link links to something else

@wraithgar wraithgar mentioned this issue Mar 17, 2022
Merged
@wraithgar wraithgar added Priority 2 secondary priority issue and removed Needs Triage needs review for next steps labels Mar 17, 2022
@nlf nlf closed this as completed in #4580 Mar 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug thing that needs fixing Priority 2 secondary priority issue Release 7.x work is associated with a specific npm 7 release
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

docs: add foreground-scripts and ignore-scripts to commands npm/cli
3 participants
@wraithgar @hasezoey @jiaqidegege