matthewbub/lg-coffee

refactor: fs-extra command should use copySync() instead of moveSync()

Aperta

#82 aperta il 23 nov 2020

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (2 fork)auto 404
good first issuehelp wantedjavascriptnodejs

Metriche repository

Star
 (0 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Referring to the command used to build the directory on local.

https://github.com/hi-matbub/lg-coffee/blob/0b89db809a112e384f79f56b39b33c4d0a7f8135/packages/lg-init/index.js#L60-L71

Reason for this is for those who decide to save this command vs using npx.

Recreate on a local environment :wrench:

run npx path_to/lg-coffee/packages/lg-init/index.js

after completing the prompts notice the og directory has been moved out of lg-coffee/packages/lg-init/lib/templates

Possible Solution 💡

we are using fs-extra and I'm blindly betting on .copySync() being the best solution here.

Guida contributor

Direzione di ricerca
Sostituisci `moveSync()` con `copySync()` nel comando che costruisce la directory, assicurando che la directory di origine rimanga intatta.
Tech stack
javascriptnodejs
Dominio
clideveloper experience
Tipo issue
Refactoring
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptNode.jsfs extra
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75