sindresorhus/electron-timber

Add ability to also log to disk

Open

#7 ouverte le 19 févr. 2018

Voir sur GitHub
 (4 commentaires) (1 réaction) (0 assignés)JavaScript (422 stars) (9 forks)user submission
enhancementhelp wanted

Description

We can use app.getPath('logs') to get the path to the logs directory.

The person that implements this should do some research on how other log solutions do it. Regarding max size, log rotation, etc.

Guide contributeur

Stack technique
javascriptnodejs
Domaine
desktop
Type d'issue
feature
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.
half day
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
Node.js basicsElectron app.getPathFile system (fs module)
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
First, examine the existing logger implementation in the repository to understand how logging works. Then, research common log rotation strategies used by other Node.js logging libraries (e.g., winston, pino). Implement disk logging using Electron's `app.getPath('logs')` to get the logs directory. Consider adding configuration for max file size and rotation.