Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: prevent import/order to build crash #44

Merged

Conversation

alekgomes
Copy link
Contributor

Descrição

Esse pull request altera a configuração no import/order de error para warn, no .eslintrc.json

Arquivos Modificados

  • frontend/.eslintrc.json: Alteração de propriedade

Antes e Depois

Antes

O build quebrava por conta de ordem de importação.

image

Depois

Agora ele avisa que a ordem está errada mas não quebra.
image

Motivação e Contexto

Acredito que ordem de importação não é uma métrica que agride tanto a qualidade de código a ponto de quebrar o build do projeto mas pode ser importante o suficiente para ser motivo de warnings.
Além disso, pode apresentar problemas futuros com o ShadCn já que ele instala os componentes localmente e não considera essa regra em seus componentes.

Como Isso Foi Testado?

  • Testado localmente, manualmente.

Tipos de Mudanças

  • Correção de bug (mudança que não quebra a compatibilidade e corrige um problema)
  • Nova funcionalidade (mudança que não quebra a compatibilidade e adiciona uma funcionalidade)
  • Mudança que quebra a compatibilidade (correção ou funcionalidade que causa uma mudança em funcionalidades existentes)

Checklist

  • Meu código segue o estilo de código deste projeto.
  • Minha mudança requer uma mudança na documentação.
  • Eu atualizei a documentação conforme necessário.
  • Eu adicionei testes para cobrir minhas mudanças.
  • Todos os novos e antigos testes passaram.

Notas Adicionais

@mffonseca mffonseca merged commit 0d2ba2b into dedevpradev:main Aug 7, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants