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
I have a quick suggestion I would like to make which would be set up pre-commit to handle the linting. pre-commit is easy to run locally (it can be installed with a git hook to run on every commit), ties in easily with github actions and with the same configuration locally and on cicd it easier to ensure that code stays clean.
If this sounds good I am happy to add the configuration and set up the cicd part too 😄 I am primarily asking @b8raoult because I just noticed you did a commit for handling some linting fixes for black.
The text was updated successfully, but these errors were encountered:
we had a pretty hefty pre-commit flying around that I worked on.
Happy if you want to add useful parts and if you have suggestions for the CI/CD, as I haven't set that up yet.
I have a quick suggestion I would like to make which would be set up pre-commit to handle the linting.
pre-commit
is easy to run locally (it can be installed with agit
hook to run on everycommit
), ties in easily with github actions and with the same configuration locally and on cicd it easier to ensure that code stays clean.If this sounds good I am happy to add the configuration and set up the cicd part too 😄 I am primarily asking @b8raoult because I just noticed you did a commit for handling some linting fixes for
black
.The text was updated successfully, but these errors were encountered: