Details
-
Bug
-
Status: Resolved
-
Major
-
Resolution: Fixed
-
None
-
None
-
None
Description
The mesos `pre-commit` hook is currently running several linters on the source code, some of which are even dynamically installed from the internet during a commit.
This can hinder development because it also applies to local commits that are not intended to be ever published, and can quickly become annoying when rebasing old branches.
Instead, we should think about putting these hooks into a separate `support/verify-reviews.py` which would be executed when trying to post a review, since at this point the patches should be cleaned up and pass all linter checks.