#3515 new defect

The pre-commit push hook takes a long time to run

Reported by: exarkun Owned by:
Priority: normal Milestone: undecided
Component: unknown Version: n/a
Keywords: review Cc:
Launchpad Bug:

Description

On my system, 10-20 seconds. This is a noticeable disruption in workflow as by the time I am pushing I am ready to move to something new but I have to wait for this to finish before I feel safe touching my checkout.

Change History (3)

comment:1 Changed at 2020-11-19T16:00:24Z by exarkun

Currently the hook runs tox -e codechecks which runs flake8 on all source files in the checkout.

It seems like it would be good to run flake8 only on files being modified by changesets being pushed. flake8 does no cross-source-file checking so there is no way for a change in file X to introduce a flake in file Y. So checking only changed files seems safe.

comment:3 Changed at 2020-11-19T16:14:18Z by exarkun

  • Keywords review added
Note: See TracTickets for help on using tickets.