Deprecate `prettier` pre-commit hook
The prettier pre-commit hook has been deprecated and the repo has been archived since prettier made some breaking changes.
They have some alternatives here, but none of them seem particularly convenient.
I'm using
- repo: https://github.com/rbubley/mirrors-prettier
rev: "v3.3.2"
hooks:
- id: prettier
types_or: [yaml, markdown, html, css, scss, javascript] #, json]
args: [--prose-wrap=always]
this seems to be maintained for now...
The convenient thing about the above mirror is that it also only tags non-pre-release versions so that the weekly struggle of reverting the pre-commit autoupdate from the 4.x pre-release version to the latest 3.x stable release is eliminated.
Would it be okay for sp-repo-review to recommend the alternative mirror?
Or could the check for the presence of prettier in the pre-commit config be loosened up so that it passes as long as the URL matches https://github.com/*/mirrors-prettier?
We can list popular mirrors, and eventually start recommending one. #460 makes this easier.
Done in both #461 and fully moved in #476.