sindresorhus/execa

Attach `.nodeChildProcess` to the execa promise

Chiusa

#413 aperta il 16 feb 2020

Vedi su GitHubGuida contributor
 (13 commenti) (1 reazione) (0 assegnatari)JavaScript (219 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (6091 stelle)
Metriche merge PR
 (Merge medio 6g 18h) (6 PR mergiate in 30 g)

Descrizione

Node.js does this now for childProcess. when you promisify it.

https://nodejs.org/api/child_process.html#child_process_child_process_exec_command_options_callback

The name is to be decided.

One day, I hope we can get rid of execa() being both a childProcess and promise and just have the .child property on the promise. But that's a huge breaking change. For now, the best we can do is to add it, and recommend new users to use it instead of the old way. We could even un-document the old way.

@ehmicky Thoughts?

Guida contributor

Direzione di ricerca
L'issue propone di aggiungere una proprietà `.nodeChildProcess` alla promise di execa, simile a child process di Node.js. I punti chiave di discussione includono la denominazione e la graduale deprecazione del vecchio comportamento duale. Esamina il codice sorgente di execa in index.js e main.js per comprendere l'implementazione corrente della promise. Rivedi la documentazione di Node.js per la callback di `child process.exec`. Considera i commenti del maintainer (specialmente @ehmicky) per le decisioni di progettazione. L'obiettivo è aggiungere la proprietà senza modifiche di rottura, potenzialmente segnando il vecchio approccio come deprecato. Ulteriori letture: discussione dell'issue collegata e eventuali PR correlati.
Tech stack
javascriptnodejs
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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.jsJavaScriptPromises
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60