matthewbub/lg-coffee

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

Offen

#82 geöffnet am 23.11.2020

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (2 Forks)auto 404
good first issuehelp wantedjavascriptnodejs

Repository-Metriken

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

Beschreibung

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.

Contributor Guide

Research-Richtung
Ersetze `moveSync()` durch `copySync()` im Befehl, der das Verzeichnis erstellt, und stelle sicher, dass das Quellverzeichnis erhalten bleibt.
Tech Stack
javascriptnodejs
Domain
clideveloper experience
Issue Type
Refactoring
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.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptNode.jsfs extra
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75