Replace pretty-quick by a different tool

[rioug]

What we should change and why (this is tech debt)

As first noted here #11165 (comment) pretty-quick is not maintained anymore.
So we should replace it by something else, maybe https:/lint-staged/lint-staged is was mentioned a few times here : prettier/pretty-quick#164

In the mean time we should not upgrade prettier

Context

This came up here #11165

Impact and timeline

[mkllnk]

I don't think that we should install pre-commit hooks by default. It should be up to the developer when to run linters. And we have CI as last line of defense. You can integrate with your IDE or you may only change the README file and don't need to run any validations.

[rioug]

The pre commit hook was added here : e0b4022
But I agree it should be part of CI

[dacook]

Something on my list of things to do when I have time (retire?) was to create a smart hook that only runs relevant linters depending on what files are changed. Also specs (eg rspec for *_spec.rb, jest for *_test.js, etc).

But that's just my personal preference; I agree that it should be user-configurable. Maybe that's what Husky is for?
looks up husky
Hmm I'm not sure what Husky is for exactly..

[binarygit]

I opened this #11824 which replaced pretty-quick with lint-staged and, had configured lint-staged to only run the liner for js, json files which are staged.

[SerenModz21]

Hi! I came across thie repo in the pretty-quick pr (prettier/pretty-quick#165) and would just like to let you know that @JounQin has taken over the project and has released pretty-quick v4, which adds support for prettier v3.

related comments:

• pretty-quick will break with prettier v3 prettier/pretty-quick#164 (comment)

  I've just finished my promise.

  pretty-quick v4 has just been released with prettier v3 support! Thanks @SukkaW for helping as always! ❤️

• feat!: support prettier 3.0 prettier/pretty-quick#165 (comment)

  I'll take over @azz's place and help to maintain this package in few days. 🍻