Introduce pre-commit / make lint
#39
Conversation
Also make the linters part of the github actions.
| @@ -0,0 +1,32 @@ | |||
| --- | |||
There was a problem hiding this comment.
(nitpick) this probably is better suited for the next commit, i.e. this one cleans up the existing code to be "lint" ready, the next "adds the linting" but it seems this file fits better into the part(?)
| @@ -0,0 +1,2 @@ | |||
| [FORMAT] | |||
There was a problem hiding this comment.
(nitpick) this probably is better suited for the next commit, i.e. this one cleans up the existing code to be "lint" ready, the next "adds the linting" but it seems this file fits better into the part(?)
| @@ -0,0 +1,10 @@ | |||
| --- | |||
There was a problem hiding this comment.
(nitpick) this probably is better suited for the next commit, i.e. this one cleans up the existing code to be "lint" ready, the next "adds the linting" but it seems this file fits better into the part(?)
| @@ -0,0 +1,2 @@ | |||
| [pep8] | |||
There was a problem hiding this comment.
(nitpick) this probably is better suited for the next commit, i.e. this one cleans up the existing code to be "lint" ready, the next "adds the linting" but it seems this file fits better into the part(?)
| @@ -0,0 +1,5 @@ | |||
| [MESSAGES CONTROL] | |||
There was a problem hiding this comment.
(nitpick) this probably is better suited for the next commit, i.e. this one cleans up the existing code to be "lint" ready, the next "adds the linting" but it seems this file fits better into the part(?)
| @@ -0,0 +1,4 @@ | |||
| [MESSAGES CONTROL] | |||
There was a problem hiding this comment.
(nitpick) this probably is better suited for the next commit, i.e. this one cleans up the existing code to be "lint" ready, the next "adds the linting" but it seems this file fits better into the part(?)
Introduce pre-commit linting with
make lint.Also add this to github actions.