Code Refactoring: Decoupling / Pass process.args as parameters
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 45/100
- Type d'issue
- Refactorisation
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- sql, typescript
- Domaine
- cli
Piste de recherche
Commencez par localiser les définitions des fonctions writeSQL et readCSV et retracer leurs utilisations de process.args. Transmettez plutôt les valeurs CLI nécessaires à ces fonctions, puis vérifiez que leur comportement de sortie est inchangé et que les paramètres des fonctions priment sur les arguments CLI.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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...)
- Langage dominant
- TypeScript
- Étoiles
- 120
- Forks
- 123
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de gitdagray/csv-to-sql-insert
-
README file improvement.Peut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 68/100
gitdagray/csv-to-sql-insert#66 · 2 commentaires ·
-
good first contributionOuverte
Difficulté 3/5 1-2 jours Accessibilité débutants 25/100
gitdagray/csv-to-sql-insert#87 · 1 commentaire ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 48/100
-
[feat 🔥] Multi-telended InsertOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 20/100
-
Add CLI progress indicatorOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
gitdagray/csv-to-sql-insert#74 · 1 commentaire ·
Toutes les issues de gitdagray/csv-to-sql-insert
Issues similaires
-
engine-rs P3 review-deferral wasm
Difficulté 2/5 1-3 heures Accessibilité débutants 77/100
maniator/verticopolis#1154 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
Les mainteneurs répondent en général sous 1 jour
-
Bug
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
telerik/kendo-react#3861 ·
Les mainteneurs répondent en général sous 1 jour
-
[bug]: use-mobile.ts uses pixel logic, causing UI state de-sync with Tailwind CSS rem breakpointsOuvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
task
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100