Consider catching thrown errors from validators
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 5/5
- Tempo estimado
- Mais de uma semana
- Facilidade para iniciantes
- 30/100
- Tipo de issue
- Funcionalidade
- Clareza
- Precisa de esclarecimento
- Status de atividade
- Ativa
- Stack de tecnologia
- node.js, typescript
- Domínio
- cli
Direção de pesquisa
A issue não nomeia arquivos, testes nem um ponto de entrada. Primeiro, esclareça se os erros lançados pelo validator devem ser capturados e como devem ser informados pela CLI; depois, pondere isso em relação a ocultar bugs do validator. Para considerar o trabalho concluído, é necessário um comportamento acordado e testes cobrindo tanto uma falha de validação tratada de forma adequada quanto exceções não intencionais.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
We currently do not catch thrown errors from validators since it is expected that you return an Error instead.
This is by design and is fine. It may be worth handling uncaught exceptions too, though, just so the CLI gracefully fails validation rather than exiting.
this is the validation implementor's fault at the end of the day, if it happens, but covering it up could give better DX.
thoughts?
the negative is that it could cover up unintentional errors. today it'll be very obvious the validator has a bug and that wouldn't be the case anymore.
- Linguagem predominante
- TypeScript
- Estrelas
- 8.1k
- Forks
- 217
- Merge médio
- 1d 23h
- PRs com merge (30d)
- 6
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um 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 bombshell-dev/clack
-
[Bug]Abertabug
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 10/100
bombshell-dev/clack#612 ·
-
rfc: accessible modeAbertaenhancement
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 35/100
bombshell-dev/clack#585 · 1 comentário · 1 reação ·
-
bug
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 68/100
bombshell-dev/clack#573 · 1 comentário · 2 reações ·
-
enhancement
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 62/100
bombshell-dev/clack#565 · 1 comentário · 1 reação ·
-
enhancement
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 48/100
bombshell-dev/clack#564 · 1 reação ·
Todas as issues de bombshell-dev/clack
Issues semelhantes
-
Mend: dependency security vulnerability untriaged
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
opensearch-project/security-dashboards-plugin#2545 ·
Mantenedores costumam responder em até 1 dia
-
Add: Dream TR SDAbertacheck:passed streams:add
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
Mantenedores costumam responder em até 1 dia
-
doctor integrity sample scans soft-deleted pages on Postgres (batch path has no deleted_at filter)Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 72/100
SocialGouv/egapro#4672 · 1 comentário ·
Mantenedores costumam responder em até 2 dias
-
area:agents area:tui bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 74/100
anthropics/claude-code#98358 ·
Mantenedores costumam responder em até 1 dia