remixz/publish-release

Cope better with `EDITOR` not being defined

Open

#6 ouverte le 17 août 2015

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (17 forks)github user discovery
enhancementhelp wanted

Métriques du dépôt

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

Description

There are a couple of ways this could be achieved:

  1. Fail-fast with a nice error message if EDITOR has not been defined.
  2. Ask the developer to enter the path to their editor in lieu of EDITOR not being defined.

Ideally, my preference would be for option 2, as I plan to use publish-release within the postpublish hook of all our modules, and this would allow developers who run npm publish to still have things work, rather than requiring them to define EDITOR, then re-run npm run postpublish.

Guide contributeur

Direction de recherche
Examinez comment EDITOR est utilisé dans la base de code et décidez de la méthode de repli préférée (message d'erreur ou invite interactive).
Stack technique
javascript
Domaine
clideveloper experience
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.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
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.
Claire
Prérequis
Node.jsGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80