yarnpkg/yarn

Command-specific flags `--ignore-scripts` shouldn't be shown with `yarn help`

Open

#2735 aperta il 20 feb 2017

Vedi su GitHub
 (10 commenti) (8 reazioni) (0 assegnatari)JavaScript (2731 fork)batch import
cat-buggood first issuehelp wantedtriaged

Metriche repository

Star
 (41.514 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Do you want to request a feature or report a bug? bug

What is the current behavior? yarn [command] --ignore-scripts does not ignore scripts

If the current behavior is a bug, please provide the steps to reproduce.

package.json:

{
  "scripts" : {
    "prebuild": "echo prebuild",
    "build": "echo build"
  }
}

$ yarn run build --ignore-scripts
yarn run v0.20.3
$ echo prebuild
prebuild
$ echo build
build
✨  Done in 0.25s.

What is the expected behavior?

$ yarn run build --ignore-scripts
yarn run v0.20.3
$ echo build
build
warning Ignored scripts due to flag.
✨  Done in 0.25s.

Please mention your node.js, yarn and operating system version.

what version
node -v 6.9.1
yarn -V 0.20.3
os-version: OS X 10.11.6

Guida contributor

Direzione di ricerca
Il problema riguarda yarn v1, che ora è congelato sul ramo principale. Per correggere il bug, bisognerebbe esaminare l'implementazione del comando `yarn run`, probabilmente in `src/cli/commands/run.js`, per vedere come viene gestito il flag ` ignore scripts`. Il flag è definito globalmente ma dovrebbe essere limitato ai comandi di tipo install. La correzione comporterebbe il filtraggio del flag dall'output di aiuto e l'assicurarsi che si applichi correttamente a `yarn run`. Rivedere i commenti sull'issue per eventuali approfondimenti o lavori già svolti.
Tech stack
javascript
Dominio
cli
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70