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 3 commits 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.

@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