Code Refactoring: Decoupling / Pass process.args as parameters
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 45/100
- Tipo de issue
- Refatoração
- Clareza
- Razoavelmente clara
- Status de atividade
- Estagnada
- Stack de tecnologia
- sql, typescript
- Domínio
- cli
Direção de pesquisa
Comece localizando as definições das funções writeSQL e readCSV e rastreando seus usos de process.args. Passe os valores CLI necessários para essas funções, em vez disso, e então verifique se o comportamento de saída delas permanece inalterado e se os parâmetros das funções têm precedência sobre os argumentos CLI.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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...)
- Linguagem predominante
- TypeScript
- Estrelas
- 121
- Forks
- 122
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de gitdagray/csv-to-sql-insert
-
README file improvement.Talvez já em andamento Um pull request vinculado a esta issue está aberto ou já foi mesclado. Aberta
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 68/100
gitdagray/csv-to-sql-insert#66 · 2 comentários ·
-
good first contributionAberta
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 25/100
gitdagray/csv-to-sql-insert#87 · 1 comentário ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 48/100
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 20/100
-
Add CLI progress indicatorAberta
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 25/100
gitdagray/csv-to-sql-insert#74 · 1 comentário ·
Todas as issues de gitdagray/csv-to-sql-insert
Issues semelhantes
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 82/100
Mantenedores costumam responder em até 1 dia
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
juice-shop/juice-shop#3706 ·
Mantenedores costumam responder em até 1 dia
-
sbx up from a subfolder silently pushes the whole work treeTalvez já em andamento @pdettori assumiu hoje. Abertaenhancement
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
Mantenedores costumam responder em até 1 dia
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
outline/outline#14348 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
bug needs-triage
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
Mantenedores costumam responder em até 1 dia