yarnpkg/yarn

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

Open

#2 735 ouverte le 20 févr. 2017

Voir sur GitHub
 (10 commentaires) (8 réactions) (0 assignés)JavaScript (2 731 forks)batch import
cat-buggood first issuehelp wantedtriaged

Métriques du dépôt

Stars
 (41 514 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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

Guide contributeur

Direction de recherche
Le problème concerne yarn v1, qui est désormais gelé sur la branche principale. Pour corriger le bug, il faudrait examiner l'implémentation de la commande `yarn run`, probablement dans `src/cli/commands/run.js`, pour voir comment l'option ` ignore scripts` est gérée. L'option est définie globalement mais devrait être limitée aux commandes de type installation. La correction impliquerait de filtrer l'option de la sortie d'aide et de s'assurer qu'elle s'applique correctement à `yarn run`. Consultez les commentaires sur l'issue pour toute information supplémentaire ou travail déjà effectué.
Stack technique
javascript
Domaine
cli
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitNode.js
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70