openpilot_comma/.pre-commit-config.yaml

repos:
-   repo: meta
    hooks:
    -   id: check-hooks-apply
    -   id: check-useless-excludes
-   repo: https://github.com/pre-commit/pre-commit-hooks
    rev: v4.1.0
    hooks:
    -   id: check-ast
        exclude: '^(pyextra)/'
    -   id: check-json
    -   id: check-xml
    -   id: check-yaml
    -   id: check-merge-conflict
    -   id: check-symlinks
    -   id: check-added-large-files
        args: ['--maxkb=100']
-   repo: https://github.com/pre-commit/mirrors-mypy
    rev: v0.931
    hooks:
    -   id: mypy
        exclude: '^(pyextra/)|(cereal/)|(rednose/)|(panda/)|(laika/)|(opendbc/)|(laika_repo/)|(rednose_repo/)/'
        additional_dependencies: ['git+https://github.com/numpy/numpy-stubs', 'types-requests', 'types-atomicwrites',
                                  'types-pycurl', 'types-certifi', 'lxml']
        args:
        - --warn-unused-ignores
        - --warn-redundant-casts
        - --warn-unreachable
        #- --html-report=/home/batman/openpilot
-   repo: https://github.com/PyCQA/flake8
    rev: 4.0.1
    hooks:
    -   id: flake8
        exclude: '^(pyextra/)|(cereal/)|(rednose/)|(panda/)|(laika/)|(opendbc/)|(laika_repo/)|(rednose_repo/)|(selfdrive/debug/)/'
        additional_dependencies: ['flake8-no-implicit-concat']
        args:
        - --indent-size=2
        - --enable-extensions=NIC
        - --select=F,E112,E113,E304,E502,E701,E702,E703,E71,E72,E731,W191,W6
        - --statistics
        - -j4
-   repo: local
    hooks:
    -   id: pylint
        name: pylint
        entry: pylint
        language: system
        types: [python]
        exclude: '^(pyextra/)|(cereal/)|(rednose/)|(panda/)|(laika/)|(laika_repo/)|(rednose_repo/)'
-   repo: local
    hooks:
    -   id: cppcheck
        name: cppcheck
        entry: cppcheck
        language: system
        types: [c++]
        exclude: '^(third_party/)|(pyextra/)|(cereal/)|(opendbc/)|(panda/)|(tools/)|(selfdrive/modeld/thneed/debug/)|(selfdrive/modeld/test/)|(selfdrive/camerad/test/)/|(installer/)'
        args:
        - --error-exitcode=1
        - --language=c++
        - --quiet
        - --force
        - -j8
pre-commit pylint (#1580) * "The commit-hook project sounds interesting though. I would definitely merge something that runs flake8 and pylint on the modified files!" - pd0wm, https://github.com/commaai/openpilot/pull/1575#issuecomment-634974344 * add pylint to pre-commit and make everything pass * Remove uncommented stuff Co-authored-by: J <user@4800.lan> old-commit-hash: bd0643424357864287ae0485da03f66df85e1fb1 5 years ago			`repos:`
linter: don't allow implicitly concatenated strings (#23513) * linter: don't allow implicitly concatenated strings * fix hyundai old-commit-hash: 2c94f155840b49639b8bf1ec996d99e268fa9a52 3 years ago			`- repo: meta`
			`hooks:`
			`- id: check-hooks-apply`
			`- id: check-useless-excludes`
pre-commit pylint (#1580) * "The commit-hook project sounds interesting though. I would definitely merge something that runs flake8 and pylint on the modified files!" - pd0wm, https://github.com/commaai/openpilot/pull/1575#issuecomment-634974344 * add pylint to pre-commit and make everything pass * Remove uncommented stuff Co-authored-by: J <user@4800.lan> old-commit-hash: bd0643424357864287ae0485da03f66df85e1fb1 5 years ago			`- repo: https://github.com/pre-commit/pre-commit-hooks`
add pre-commit hook for large files (#23505) old-commit-hash: ee52a61c7d5289d8dbfbd27558177f5757929325 3 years ago			`rev: v4.1.0`
pre-commit pylint (#1580) * "The commit-hook project sounds interesting though. I would definitely merge something that runs flake8 and pylint on the modified files!" - pd0wm, https://github.com/commaai/openpilot/pull/1575#issuecomment-634974344 * add pylint to pre-commit and make everything pass * Remove uncommented stuff Co-authored-by: J <user@4800.lan> old-commit-hash: bd0643424357864287ae0485da03f66df85e1fb1 5 years ago			`hooks:`
			`- id: check-ast`
acados support (#21319) * acados support * add pyextra to python path * x86 libs old-commit-hash: 6e4c2aa4d286bb43f091ad13820f564f70e4c0d7 4 years ago			`exclude: '^(pyextra)/'`
pre-commit pylint (#1580) * "The commit-hook project sounds interesting though. I would definitely merge something that runs flake8 and pylint on the modified files!" - pd0wm, https://github.com/commaai/openpilot/pull/1575#issuecomment-634974344 * add pylint to pre-commit and make everything pass * Remove uncommented stuff Co-authored-by: J <user@4800.lan> old-commit-hash: bd0643424357864287ae0485da03f66df85e1fb1 5 years ago			`- id: check-json`
			`- id: check-xml`
			`- id: check-yaml`
Add pre-commit hooks (#1629) old-commit-hash: ab83e48ec4f7c7ddaa742d9797b0d38646fdb268 5 years ago			`- id: check-merge-conflict`
			`- id: check-symlinks`
add pre-commit hook for large files (#23505) old-commit-hash: ee52a61c7d5289d8dbfbd27558177f5757929325 3 years ago			`- id: check-added-large-files`
			`args: ['--maxkb=100']`
Run mypy commit hook (#1591) * run mypy commit hook * fix mypy errors old-commit-hash: 3d08dcc3b27936cb14c0eae63605be9a6c077380 5 years ago			`- repo: https://github.com/pre-commit/mirrors-mypy`
add pre-commit hook for large files (#23505) old-commit-hash: ee52a61c7d5289d8dbfbd27558177f5757929325 3 years ago			`rev: v0.931`
Run mypy commit hook (#1591) * run mypy commit hook * fix mypy errors old-commit-hash: 3d08dcc3b27936cb14c0eae63605be9a6c077380 5 years ago			`hooks:`
			`- id: mypy`
pre-commit: fix exclude patterns (#23375) old-commit-hash: a1a9a0f342bc40540d31d1cfbc10c6263a748ef3 3 years ago			`exclude: '^(pyextra/)\|(cereal/)\|(rednose/)\|(panda/)\|(laika/)\|(opendbc/)\|(laika_repo/)\|(rednose_repo/)/'`
update pre-commit hook versions (#22526) * update pre-commit hook versions * fix mypy old-commit-hash: a255449539b690f2bc16fbdf4d3a27297451dcb0 4 years ago			`additional_dependencies: ['git+https://github.com/numpy/numpy-stubs', 'types-requests', 'types-atomicwrites',`
lines for mypy report old-commit-hash: 39cf8624355ae30c3b90b4c8d56ba44b665dda10 3 years ago			`'types-pycurl', 'types-certifi', 'lxml']`
mypy: enable --warn-unreachable (#24282) * Optional solves mypy assuming None is always None * oops * spaces * Update registration.py * fix local error Co-authored-by: Willem Melching <willem.melching@gmail.com> old-commit-hash: b8b15d5cf1d2507c89bbe0e3b1de1ff043d9078e 3 years ago			`args:`
mypy: enable --warn-unused-ignores (#24287) * this is the easy part to check * Not sure what this fixes, https://github.com/commaai/openpilot/pull/24282/commits/8aaf00c6a41ac4c9c5e56f8a2dfed8591f153b67 * bump laika * works here * cleanup * add types-certifi Co-authored-by: Adeeb Shihadeh <adeebshihadeh@gmail.com> old-commit-hash: 049b163ea0494a21956d89d4cb2a527dcbd62c0c 3 years ago			`- --warn-unused-ignores`
Add --warn-redundant-casts to mypy checks (#24304) * add mypy checks for redundant-casts and unused-ignores * revert unnecessary changes * ignore Co-authored-by: Adeeb Shihadeh <adeebshihadeh@gmail.com> old-commit-hash: ee0021b86cc6ae6a9105d82798c34c5371992afb 3 years ago			`- --warn-redundant-casts`
mypy: enable --warn-unreachable (#24282) * Optional solves mypy assuming None is always None * oops * spaces * Update registration.py * fix local error Co-authored-by: Willem Melching <willem.melching@gmail.com> old-commit-hash: b8b15d5cf1d2507c89bbe0e3b1de1ff043d9078e 3 years ago			`- --warn-unreachable`
lines for mypy report old-commit-hash: 39cf8624355ae30c3b90b4c8d56ba44b665dda10 3 years ago			`#- --html-report=/home/batman/openpilot`
use github url instead of gitlab mirror old-commit-hash: 266dfe5a9b521d62cfdfdca73ffe622deee82013 5 years ago			`- repo: https://github.com/PyCQA/flake8`
update pre-commit hook versions (#22526) * update pre-commit hook versions * fix mypy old-commit-hash: a255449539b690f2bc16fbdf4d3a27297451dcb0 4 years ago			`rev: 4.0.1`
flake8 in pre-commit(#1583) old-commit-hash: 29aaa44740b5dae3c1cc0c273c72c7eb53135e5e 5 years ago			`hooks:`
			`- id: flake8`
pre-commit: fix exclude patterns (#23375) old-commit-hash: a1a9a0f342bc40540d31d1cfbc10c6263a748ef3 3 years ago			`exclude: '^(pyextra/)\|(cereal/)\|(rednose/)\|(panda/)\|(laika/)\|(opendbc/)\|(laika_repo/)\|(rednose_repo/)\|(selfdrive/debug/)/'`
linter: don't allow implicitly concatenated strings (#23513) * linter: don't allow implicitly concatenated strings * fix hyundai old-commit-hash: 2c94f155840b49639b8bf1ec996d99e268fa9a52 3 years ago			`additional_dependencies: ['flake8-no-implicit-concat']`
flake8 in pre-commit(#1583) old-commit-hash: 29aaa44740b5dae3c1cc0c273c72c7eb53135e5e 5 years ago			`args:`
linter: don't allow implicitly concatenated strings (#23513) * linter: don't allow implicitly concatenated strings * fix hyundai old-commit-hash: 2c94f155840b49639b8bf1ec996d99e268fa9a52 3 years ago			`- --indent-size=2`
			`- --enable-extensions=NIC`
Honda cleanup (#22008) * cleanup magic numbers * long tuning * indentation * remove noqa * little more old-commit-hash: 9e03aa0b92fd846bc643040fa8b8d4ab06ee2855 4 years ago			`- --select=F,E112,E113,E304,E502,E701,E702,E703,E71,E72,E731,W191,W6`
Enable more flake8 checks (#1602) * enable some more flake8 checks * some more quick ones * bump opendbc * e401 * e711 e712 * e115 e116 * e222 * e301 * remove that * e129 * e701 e702 * e125 e131 * e227 * e306 * e262 * W503 * e713 * e704 * e731 * bump opendbc * fix some e722 old-commit-hash: d9bf9f0a4036f55411f6dfbb438990a5eb7f4930 5 years ago			`- --statistics`
linter: don't allow implicitly concatenated strings (#23513) * linter: don't allow implicitly concatenated strings * fix hyundai old-commit-hash: 2c94f155840b49639b8bf1ec996d99e268fa9a52 3 years ago			`- -j4`
pre-commit pylint (#1580) * "The commit-hook project sounds interesting though. I would definitely merge something that runs flake8 and pylint on the modified files!" - pd0wm, https://github.com/commaai/openpilot/pull/1575#issuecomment-634974344 * add pylint to pre-commit and make everything pass * Remove uncommented stuff Co-authored-by: J <user@4800.lan> old-commit-hash: bd0643424357864287ae0485da03f66df85e1fb1 5 years ago			`- repo: local`
			`hooks:`
			`- id: pylint`
			`name: pylint`
			`entry: pylint`
			`language: system`
			`types: [python]`
linter: don't allow implicitly concatenated strings (#23513) * linter: don't allow implicitly concatenated strings * fix hyundai old-commit-hash: 2c94f155840b49639b8bf1ec996d99e268fa9a52 3 years ago			`exclude: '^(pyextra/)\|(cereal/)\|(rednose/)\|(panda/)\|(laika/)\|(laika_repo/)\|(rednose_repo/)'`
Add cppcheck as a pre-commit hook (#1646) * add cppcheck as a pre-commit hook * fix Dockerfile * update precommit config * exclude panda and opendbc old-commit-hash: 3c461ca5ccf1b0ae2b9164feced96a5aa4998555 5 years ago			`- repo: local`
			`hooks:`
			`- id: cppcheck`
			`name: cppcheck`
			`entry: cppcheck`
			`language: system`
			`types: [c++]`
pre-commit: fix exclude patterns (#23375) old-commit-hash: a1a9a0f342bc40540d31d1cfbc10c6263a748ef3 3 years ago			`exclude: '^(third_party/)\|(pyextra/)\|(cereal/)\|(opendbc/)\|(panda/)\|(tools/)\|(selfdrive/modeld/thneed/debug/)\|(selfdrive/modeld/test/)\|(selfdrive/camerad/test/)/\|(installer/)'`
Add cppcheck as a pre-commit hook (#1646) * add cppcheck as a pre-commit hook * fix Dockerfile * update precommit config * exclude panda and opendbc old-commit-hash: 3c461ca5ccf1b0ae2b9164feced96a5aa4998555 5 years ago			`args:`
			`- --error-exitcode=1`
fix cppcheck on ubuntu 19.10 old-commit-hash: 5f59fd4f40fda2dc07ab140253730058726ea753 5 years ago			`- --language=c++`
Add cppcheck as a pre-commit hook (#1646) * add cppcheck as a pre-commit hook * fix Dockerfile * update precommit config * exclude panda and opendbc old-commit-hash: 3c461ca5ccf1b0ae2b9164feced96a5aa4998555 5 years ago			`- --quiet`
			`- --force`
			`- -j8`