sindresorhus/update-notifier

Run in child process instead of lazy loading every dependency

Open

#210 aperta il 26 lug 2021

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)JavaScript (164 fork)user submission
enhancementhelp wanted

Metriche repository

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

Descrizione

Not a big fan [of lazy loading], but I've seen update-notifier as a require time offender when profiling various things. The idea of update-notifier is that it should have no impact on the consumer. It's just an added bonus, not critical. https://github.com/yeoman/update-notifier/pull/82#issuecomment-225636213

I may be wrong, it seems to me that this change was made simply to hide the package from the very initial load but in practice every dependency is still loaded right after.

update-notifier already uses a child process to do the checking, so I think it'd be best to do the whole thing there.

Related:

Guida contributor

Direzione di ricerca
Rivedi l'implementazione corrente in index.js, in particolare la chiamata al processo figlio alla riga 98. Comprendi come funziona attualmente il lazy loading esaminando le istruzioni require. Quindi, progetta una soluzione in cui l'intera logica di controllo degli aggiornamenti viene eseguita in un processo figlio, in modo che le dipendenze non vengano caricate nel processo principale. Considera test e casi limite.
Tech stack
javascriptnodejs
Dominio
cli
Tipo issue
Performance
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Node.jschild process
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60