Code Refactoring: Decoupling / Pass process.args as parameters
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 45/100
- Issue-Typ
- Refactoring
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- sql, typescript
- Bereich
- cli
Rechercherichtung
Beginne damit, die Definitionen der Funktionen writeSQL und readCSV zu lokalisieren und nachzuverfolgen, wie sie process.args verwenden. Übergebe stattdessen die erforderlichen CLI-Werte an diese Funktionen und überprüfe anschließend, dass sich ihr Ausgabeverhalten nicht geändert hat und dass Funktionsparameter Vorrang vor CLI-Argumenten haben.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Instead of depending on process.args inside both writeSQL and readCSV functions, we can simply pass necessary CLI arguments as function parameters.
This will make it easier to maintain and it will also give priority to the function arguments over the CLI argument which can be useful in case (in a future update) we want to dynamically change the output file name (ex: splitting big data into multiple files such as output_1.sql, output_2.sql...)
- Vorherrschende Sprache
- TypeScript
- Sterne
- 120
- Forks
- 123
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus gitdagray/csv-to-sql-insert
-
README file improvement.Offen
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 68/100
gitdagray/csv-to-sql-insert#66 · 2 Kommentare ·
-
good first contributionOffen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 25/100
gitdagray/csv-to-sql-insert#87 · 1 Kommentar ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 48/100
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 20/100
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100
gitdagray/csv-to-sql-insert#74 · 1 Kommentar ·
Alle Issues in gitdagray/csv-to-sql-insert
Ähnliche Issues
-
bug via-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
pingdotgg/t3code#14452 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
solana-foundation/program-examples#747 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 9 Tagen
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
remotion-dev/remotion#11847 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
openwatersio/slackwater#355 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
melgarafael/DeskcommCRM#1998 · 3 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag