ryansurf / cli-surf

Get surf and ocean data from the command line interface
7 stars 10 forks source link

Introducing pre-commit hooks #29

Closed K-dash closed 1 month ago

K-dash commented 1 month ago

@ryansurf I've been noticing that our CI keeps failing whenever we forget to run the Linter/Formatter before committing our code.

https://github.com/ryansurf/cli-surf/pull/28/commits/85582c8a3eabcc9653380fe60461ff3e5d32b445

So, I was thinking, why don't we introduce pre-commit hooks to our workflow? That way, the Linter/Formatter will run automatically every time we commit, ensuring that our code is always in tip-top shape!

f you're on board with this idea, just give me a shout, and I'll be more than happy to set it up 🙌

ryansurf commented 1 month ago

I actually had this set up for the black formatter but forgot to commit the code, deleted the repo and now i dont have it :expressionless:

Im on board, lets do it!

K-dash commented 1 month ago

Happy to help! I'll get right on it 👍