Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding pre-commit #338

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Adding pre-commit #338

wants to merge 1 commit into from

Commits on Sep 3, 2020

  1. Adding pre-commit

    Introduced changes:
    - pre-commit config and rules
    - Flake8 is covered in the pre-commit hooks.
    - Applying fixes for pre-commit compliance in all code.
    
    Also it is worth to note that commit hash will be used instead of version tags
    in pre-commit to prevend arbitrary code from running in developer's machines.
    
    pre-commit will be used to:
    - trailing whitespace;
    - Replaces or checks mixed line ending (mixed-line-ending);
    - Forbid files which have a UTF-8 byte-order marker (check-byte-order-marker);
    - Checks that non-binary executables have a proper
      shebang (check-executables-have-shebangs);
    - Check for files that contain merge conflict strings (check-merge-conflict);
    - Check for debugger imports and py37+ breakpoint()
      calls in python source (debug-statements);
    - Attempts to load all yaml files to verify syntax (check-yaml);
    - Run flake8 checks (flake8) (local)
    
    For further details about tests please refer to:
    https://github.com/pre-commit/pre-commit-hooks
    4383 committed Sep 3, 2020
    Configuration menu
    Copy the full SHA
    885d2b5 View commit details
    Browse the repository at this point in the history