vadimdemedes/gifi

Migrate to term-img module

Open

#27 aperta il 9 mar 2016

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (42 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (1025 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Would be way better to use a module (https://github.com/sindresorhus/term-img) for outputting GIFs, rather than spawning a new process and piping data from it.

Guida contributor

Direzione di ricerca
Il problema suggerisce di sostituire l'implementazione corrente dell'output GIF (che genera un processo figlio e invia i dati tramite pipe) con il modulo npm 'term img'. Innanzitutto, esamina il codice attuale in gifi, probabilmente nello script principale dove viene utilizzato child process.spawn per l'output dei GIF. Quindi, installa ed esplora il modulo term img (https://github.com/sindresorhus/term img) per comprenderne l'API. La migrazione comporta la sostituzione della logica di spawn con la funzione di rendering di term img. Assicurati che il percorso del file GIF venga passato correttamente e che il modulo gestisca correttamente l'output del terminale. Non sono necessarie dipendenze aggiuntive o logiche complesse, rendendo questo un refactoring semplice.
Tech stack
javascript
Dominio
cli
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptNode.jschild process
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75