yarnpkg/yarn

Workspaces don't run lifecycle scripts when linking local packages

Open

#3911 aperta il 11 lug 2017

Vedi su GitHub
 (36 commenti) (58 reazioni) (0 assegnatari)JavaScript (2731 fork)batch import
cat-buggood first issuehelp wantedhigh-prioritytriaged

Metriche repository

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

Descrizione

What is the current behavior?

When you do a yarn install in a workspaces enabled project, yarn doesn't run the various lifecycle scripts for workspace packages, e.g. prepare prepublish preinstall etc

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

Here is a commit adding a failing test case: https://github.com/jquense/yarn/commit/61eaf4239f707f772af5713ebc98963589a6dad4

What is the expected behavior?

Lifecycle scripts are run for the local packages so they have an opportunity to build steps. This is what lerna does.

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

yarn 27.5 node 8 Mac Sierra

cc @bestander

Guida contributor

Direzione di ricerca
Rivedi il test fallito su https://github.com/jquense/yarn/commit/61eaf4239f707f772af5713ebc98963589a6dad4 per comprendere il comportamento atteso. Esamina la logica di collegamento delle workspace nel codice sorgente di yarn, in particolare dove dovrebbero essere chiamati gli script del ciclo di vita per i pacchetti collegati. Confronta con l'approccio di lerna in BootstrapCommand. Poiché yarn 1.x è congelato, valuta se la correzione dovrebbe essere proposta per il repository berry.
Tech stack
javascript
Dominio
clitooling
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
GitJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50