sindresorhus/execa

Attach `.nodeChildProcess` to the execa promise

Fermée

#413 ouverte le 16 févr. 2020

Voir sur GitHubGuide contributeur
 (13 commentaires) (1 réaction) (0 personne assignée)JavaScript (219 forks)batch import
enhancementhelp wanted

Métriques du dépôt

Stars
 (6 091 étoiles)
Métriques de merge PR
 (Merge moyen 6j 18h) (6 PRs mergées en 30 j)

Description

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?

Guide contributeur

Direction de recherche
L'issue propose d'ajouter une propriété `.nodeChildProcess` à la promesse execa, similaire à child process de Node.js. Les points clés de discussion incluent la dénomination et la dépréciation progressive de l'ancien comportement double. Examinez le code source d'execa dans index.js et main.js pour comprendre l'implémentation actuelle de la promesse. Consultez la documentation de Node.js pour le callback `child process.exec`. Prenez en compte les commentaires du mainteneur (en particulier @ehmicky) pour les décisions de conception. L'objectif est d'ajouter la propriété sans changements cassants, en marquant potentiellement l'ancienne approche comme obsolète. Lectures complémentaires : discussion de l'issue liée et tout PR associé.
Stack technique
javascriptnodejs
Domaine
backend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Node.jsJavaScriptPromises
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60