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

ci: switch pretty-quick with lint-staged #68

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mstrk
Copy link
Contributor

@mstrk mstrk commented Nov 20, 2023

Este PR troca pretty-quick por lint-staged.

Considerações:

  • A versão do prettier foi atualizada para latest
  • Foi preciso fazer update dos packages de eslint também por causa da versão do prettier. Não era preciso todos, mas por boas práticas foi feito em todos.
  • Foi adicionado a flag --fix para resolver problemas que possam ser facilmente resolvidos pelo eslint.
  • Achei por bem fazer um loose às regras de eslint para dar só um warning nas questões relacionadas com formatação. Se já temos o prettier a formatar o código no pre-commit hook acho por bem deixar o developer "codar" como quiser. Mas se for preciso mesmo mostrar um error on development time é só reverter a change na regra.

close #67

@mstrk
Copy link
Contributor Author

mstrk commented Nov 20, 2023

Parece-me que já existiam erros de prettier na main branch ou os defaults do prettier mudaram. Eu posso resolver os mesmos neste PR, ou outro PR deve ser feito com os erros do check resolvidos, antes de se fazer merge deste.

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.

pretty-quick deixa de funcionar com prettier v3
1 participant