aquasecurity / kube-hunter

Hunt for security weaknesses in Kubernetes clusters
Apache License 2.0
4.75k stars 585 forks source link

Full linting standards #320

Closed iyehuda closed 4 years ago

iyehuda commented 4 years ago

What would you like to be added

Use full flake8 functionality alongside black. Use pre-commit for creating a pre-commit hook using black.

Why is this needed

Currently we do a minimal linting check in CI and don't enforce code standards. Taking advantage of all of flake8's capabilities can help preventing bugs that weren't catched in code review/tests. Use black to save code review time spent on linting comments. Using a pre-commit hook to run black for auto fixing can also save time for developers.