aspiers/github-notifications

make constants configurable

Aperta

#1 aperta il 27 feb 2017

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)Ruby (1 fork)github user discovery
enhancementhelp wanted

Metriche repository

Star
 (2 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

It would be better if these constants were all configurable via a YAML or .ini file:

SLEEP_INTERVAL = 5 * 60 # 5 minutes
ONLINE_CHECK_TARGET_HOST = "8.8.8.8"
PING_COUNT = 10
LOG_DIR = ENV['HOME'] + "/.log"
LOG_FILE = LOG_DIR + "/github-notifications.log"

Guida contributor

Direzione di ricerca
Esamina le costanti esistenti nel codice sorgente, quindi implementa un parser per file di configurazione YAML o INI e sostituisci i valori hardcoded con valori configurabili.
Tech stack
ruby
Dominio
cli
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Ruby
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75