You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
While going over our Docs for Development Installation, I didn't find any mention of pre-commit hooks.
They are to be considered a mandatory part of development (as they are enforced in CI builds) and it's a big disadvantage to miss them in a local setup.
The text was updated successfully, but these errors were encountered:
I think it's important to mention this in the contributing guide, for sure. IIRC, we didn't mention this when we started using pre-commit because we weren't 100% sure we will be adopting this pattern. However, I think we are already all on board with the current pattern and we are ready to document it so contributors also run pre-commit on their own PRs.
@humitos
Would you have any recommandation on where is a good location to place this documentation about pre-commit among current menus ?
So that opening a PR would be easier for this current issue.
While going over our Docs for Development Installation, I didn't find any mention of pre-commit hooks.
They are to be considered a mandatory part of development (as they are enforced in CI builds) and it's a big disadvantage to miss them in a local setup.
The text was updated successfully, but these errors were encountered: