web-miner icon indicating copy to clipboard operation
web-miner copied to clipboard

Add pre-push hooks

Open Createdd opened this issue 7 years ago • 2 comments

Add pre-push hook to check for:

  • linting
  • formatting
  • tests

Yes, this is done in Travis, but it's good to have immediate feedback before pushing bad code

Createdd avatar Jul 26 '18 13:07 Createdd

Good article here: https://ljvmiranda921.github.io/notebook/2018/06/21/precommits-using-black-and-flake8/

Createdd avatar Aug 05 '18 14:08 Createdd

Pushed a branch with the config: https://github.com/Keep-Current/web-miner/blob/feature/21_Add_pre_push_hooks/.pre-commit-config.yaml But it's not working. Not quite sure why.

If someone else has done something similar feel free to take on that issue

Createdd avatar Aug 05 '18 14:08 Createdd