Skip to content

[pre-commit] Add a hook for prettier #752

Merged
Pierre-Sassoulas merged 1 commit intomasterfrom
add-prettier
Apr 10, 2025
Merged

[pre-commit] Add a hook for prettier #752
Pierre-Sassoulas merged 1 commit intomasterfrom
add-prettier

Conversation

@Pierre-Sassoulas
Copy link
Copy Markdown
Collaborator

@Pierre-Sassoulas Pierre-Sassoulas commented Apr 10, 2025

prettier format jinja files pretty poorly, and djlint had some issue so I removed it.

@Pierre-Sassoulas Pierre-Sassoulas changed the title [pre-commit] Add prettier and djlint [pre-commit] Add a hook for prettier Apr 10, 2025
@Pierre-Sassoulas Pierre-Sassoulas merged commit 6f35b62 into master Apr 10, 2025
7 checks passed
@Pierre-Sassoulas Pierre-Sassoulas deleted the add-prettier branch April 10, 2025 13:39
@carlio
Copy link
Copy Markdown
Member

carlio commented Apr 10, 2025

I removed prettier a long time ago because I got into a fight with my local node version and I didn't like having to have node in order to write Python :(

@Pierre-Sassoulas
Copy link
Copy Markdown
Collaborator Author

It should be confined in the pre-commit environnement here (first install is a little slow for sure). Let's revert if you had issues with pre-commit and prettier together :)

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 this pull request may close these issues.

3 participants