Skip to content

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.

@IvoPereira
Copy link
Member

@mstrk nunca recebi a notificação deste PR e só agora o vi. Queres ainda dar merge dele? Se sim, consegues só verificar os conflitos pf?

@mstrk
Copy link
Contributor Author

mstrk commented Feb 21, 2025

@IvoPereira Também já não me lembro deste PR. Deixa-me dar uma vista de olhos e perceber se ainda faz sentido. Se fizer atualizo e faço-te ping 👍

Edit: dou uma vista de olhos amanhã.

@mstrk
Copy link
Contributor Author

mstrk commented Feb 22, 2025

@IvoPereira podem rever o PR. Parece estar tudo a funcionar como é suposto; deixei a rule endOfLine que foi introduzida no #76 porque não faz diferença para este PR mas pode fazer para vocês.

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

2 participants