ci: switch pretty-quick with lint-staged#68
Conversation
|
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 |
|
@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? |
|
@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ã. |
|
@IvoPereira podem rever o PR. Parece estar tudo a funcionar como é suposto; deixei a rule |
|
Hello @mstrk, entretanto atualizei o main para Node 20.19, Discordjs 14 e Prettier 3. Podes dar rebase no teu PR com main pf? |
Este PR troca
pretty-quickporlint-staged.Considerações:
prettierfoi atualizada para latesteslinttambém por causa da versão doprettier. Não era preciso todos, mas por boas práticas foi feito em todos.--fixpara resolver problemas que possam ser facilmente resolvidos peloeslint.eslintpara dar só um warning nas questões relacionadas com formatação. Se já temos oprettiera 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