saberland/saber

Show a notifier when it's required to restart server

Offen

#509 geöffnet am 08.10.2019

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (1 Reaktion) (0 zugewiesene Personen)JavaScript (127 Forks)batch import
contribution welcomehacktoberfesttype: feature

Repository-Metriken

Stars
 (2.151 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuchen Sie die referenzierte Datei `packages/saber/lib/plugins/extend node api.js` und verstehen Sie, wie Sie mit dem Paket `node notifier` eine Benachrichtigung hinzufügen. Erwägen Sie, den Benachrichtiger im Paket `saber utils` bereitzustellen und auszulösen, wenn Konfigurationsdateien geändert werden.
Tech Stack
javascriptvuenodejs
Domain
developer experiencetooling
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Node.jsnpm
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65