saberland/saber

Show a notifier when it's required to restart server

Ouverte

#509 ouverte le 8 oct. 2019

Voir sur GitHubGuide contributeur
 (3 commentaires) (1 réaction) (0 personne assignée)JavaScript (127 forks)batch import
contribution welcomehacktoberfesttype: feature

Métriques du dépôt

Stars
 (2 151 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Feature request

What problem does this feature solve?

https://github.com/saberland/saber/blob/83c18a706637fedccdbe0ad34566218dc5780d94/packages/saber/lib/plugins/extend-node-api.js#L89

People often modify saber-config.js, saber-node.js and forget to restart server.

How should this be implemented in your opinion?

Maybe use https://www.npmjs.com/package/node-notifier, and expose it in saber-utils package

Guide contributeur

Direction de recherche
Examinez le fichier référencé `packages/saber/lib/plugins/extend node api.js` et comprenez comment ajouter une notification à l'aide du package `node notifier`. Envisagez d'exposer le notificateur dans le package `saber utils` et de le déclencher lorsque les fichiers de configuration changent.
Stack technique
javascriptvuenodejs
Domaine
developer experiencetooling
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.
2
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.
Claire
Prérequis
Node.jsnpm
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65