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

doc: update directories to directory in corepack --help #55163

Closed
wants to merge 1 commit into from

Conversation

lvqq
Copy link
Contributor

@lvqq lvqq commented Sep 29, 2024

━━━ General commands ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  corepack disable [--install-directory #0] ...
    Remove the Corepack shims from the install directory

  corepack enable [--install-directory #0] ...
    Add the Corepack shims to the install directories

  corepack hydrate [--activate] <fileName>
    Import a package manager into the cache

  corepack prepare [--activate] [--all] [--json] [-o,--output] ...
    Generate a package manager archive

You can also print more details about any of these commands by calling them with
the `-h,--help` flag right after the command name.

As option --install-directory is a specified directory, so the command description should be directory

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/security-wg

@nodejs-github-bot nodejs-github-bot added dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run. labels Sep 29, 2024
@RedYetiDev RedYetiDev closed this Sep 29, 2024
@RedYetiDev
Copy link
Member

This should be upstreamed to corepack.

@RedYetiDev RedYetiDev added wrong repo Issues that should be opened in another repository. and removed needs-ci PRs that need a full CI run. labels Sep 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file. wrong repo Issues that should be opened in another repository.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants