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

pretty-quick deixa de funcionar com prettier v3 #67

Open
mstrk opened this issue Nov 20, 2023 · 2 comments · May be fixed by #68
Open

pretty-quick deixa de funcionar com prettier v3 #67

mstrk opened this issue Nov 20, 2023 · 2 comments · May be fixed by #68

Comments

@mstrk
Copy link
Contributor

mstrk commented Nov 20, 2023

O pretty-quick vai deixar de funcionar por causa de uma API deprecada no prettier v3. Mais detalhes neste issue pretty-quick/issues/164.

Isto é um problema que vai afetar o pre-commit hook neste repositório, quando o prettier for updated também.

Como não existe muita manutenção na repo do pretty-quick eu vou tentar arranjar outra solução mais mantida e após testar em outros projetos faço pull request aqui.

@mstrk
Copy link
Contributor Author

mstrk commented Nov 20, 2023

@IvoPereira experimentei o lint-staged e é "pretty quick" pun intended, sugerido no issue supracitado. É até mais robusto e melhor documentado. E parece ser mantido regularmente.

Faz sentido um pull request?

Caso seja, diz-me por favor se a ordem do test:once vs lint conta. Ou posso delegar o format e o lint para o lint-staged (que os vai correr ao mesmo tempo) e só depois correr o test:once. Também há a possibilidade de correr tudo ao mesmo tempo caso não introduza side effects.

@IvoPereira
Copy link
Member

IvoPereira commented Nov 20, 2023

@mstrk obrigado pelo heads up.

O pretty quick parece-me ok e podes enviar PR 👍 não há qualquer problema em alterar a ordem do meu ponto de vista.

Até os podemos pôr a correr em paralelo, sem problema.

@mstrk mstrk linked a pull request Nov 20, 2023 that will close this issue
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 a pull request may close this issue.

2 participants