chore: change pre-commit hook for prettier #1108
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Context
Prettier pre-commit hook is failing a bit randomly in our CI: see here where it wants to remove lines and here where it wants to add the very same lines.
What has been done
https://github.com/pre-commit/mirrors-prettier isn't maintained anymore and was archived in April 2024.
We are switching to https://github.com/rbubley/mirrors-prettier which is a maintained fork of the above. Hopefully it will fix the issue.
Also adding along the way some new standards hooks.
Validation
Can't really validate, just hoping we won't have that random thing again.
PR check list
skip-changelog
label has been added to the PR.