Commit b226eb80 authored by David Fairbrother's avatar David Fairbrother Committed by Peterson, Peter
Browse files

Fix pre-commit clang-format only parsing one file

Because of a bug in the hook we were only parsing a single arg instead
of n args.
This meant pre-commit checked the first file then discarded the rest
parent 208237d7
......@@ -4,6 +4,7 @@ default_language_version:
exclude: ^$|instrument/
repos:
# Run fast code improvement/checks before running PR specific helpers.
- repo: https://github.com/pre-commit/pre-commit-hooks.git
rev: 2618198e9658cccb4a53f04af0f7d642109f3b54
......@@ -12,13 +13,13 @@ repos:
args: [--markdown-linebreak-ext=md]
exclude: .patch$
- id: check-added-large-files
args: ["--maxkb=4096"]
args: ['--maxkb=4096']
- id: check-xml
- id: check-yaml
args: [--allow-multiple-documents]
- repo: https://github.com/mantidproject/pre-commit-hooks.git
rev: f99fd08a5f4ece5a196adeda23d20b4491cfd97b
rev: 511456a60b0c6cbb079e64a7cd8e3302f04859a6
hooks:
- id: clang-format
exclude: Testing/Tools/cxxtest|tools
......@@ -26,10 +27,11 @@ repos:
# Exclude sphinx / template file
exclude: test_doctest.py|python_export_maker.py
- repo: https://github.com/cheshirekow/cmake-format-precommit
rev: v0.6.13
hooks:
- id: cmake-format
- id: cmake-format
#- id: cmake-lint <- not ready yet
- repo: https://gitlab.com/pycqa/flake8.git
......@@ -38,4 +40,4 @@ repos:
- id: flake8
# Have to manually copy this from the .flake8 as flake8 will just ignore the config if the file is passed
exclude: Framework/PythonInterface/test|qt/applications/workbench/workbench/app/resources.py|qt/python/mantidqt/mantidqt/resources.py|qt/python/mantidqt/mantidqt/dialogs/errorreports/resources_qt4.py|qt/python/mantidqt/mantidqt/dialogs/errorreports/resources_qt5.py|Testing/PerformanceTests|Testing/SystemTests/lib|Testing/SystemTests/scripts|Testing/SystemTests/tests/framework/reference|scripts/ErrorReporter/ui_errorreport.py
args: ["--extend-ignore=E999,F821,E127"]
args: ['--extend-ignore=E999,F821,E127']
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment