From 323f463f28f793c3be80494b734fd46c9bd16064 Mon Sep 17 00:00:00 2001 From: Chris Hennes Date: Wed, 5 Mar 2025 21:04:23 -0600 Subject: [PATCH 1/2] GitHub: Add to pre-commit --- .pre-commit-config.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 4654bbb76a..5324482d6c 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -4,6 +4,7 @@ # See https://pre-commit.com/hooks.html for more hooks files: | (?x)^( + .github| src/Base| src/Main| src/Tools| From d961be1ab6ed948dad2e14549b5f6ea6ed9cafd1 Mon Sep 17 00:00:00 2001 From: Chris Hennes Date: Wed, 5 Mar 2025 21:04:33 -0600 Subject: [PATCH 2/2] GitHub: Run pre-commit on all files --- .github/codespellignore | 2 +- .github/problemMatcher/blackWarning.json | 22 +++++++-------- .github/problemMatcher/clang.json | 26 ++++++++--------- .github/problemMatcher/codespell.json | 24 ++++++++-------- .github/problemMatcher/cpplint.json | 24 ++++++++-------- .../problemMatcher/grepMatcherWarning.json | 24 ++++++++-------- .github/problemMatcher/msvc.json | 6 ++-- .github/problemMatcher/pylintError.json | 28 +++++++++---------- .github/problemMatcher/pylintWarning.json | 28 +++++++++---------- ...o-close_stale_issues_and_pull-requests.yml | 1 - 10 files changed, 93 insertions(+), 92 deletions(-) diff --git a/.github/codespellignore b/.github/codespellignore index fa5a17d758..85641cfdea 100644 --- a/.github/codespellignore +++ b/.github/codespellignore @@ -74,4 +74,4 @@ vas vertexes vew wallthickness -zuser \ No newline at end of file +zuser diff --git a/.github/problemMatcher/blackWarning.json b/.github/problemMatcher/blackWarning.json index 2d48de1576..21b355772c 100644 --- a/.github/problemMatcher/blackWarning.json +++ b/.github/problemMatcher/blackWarning.json @@ -1,15 +1,15 @@ { - "problemMatcher": [ - { - "owner": "black-warning", - "severity": "warning", - "pattern": [ + "problemMatcher": [ { - "regexp": "^(would reformat (.*))$", - "file": 2, - "message": 1 + "owner": "black-warning", + "severity": "warning", + "pattern": [ + { + "regexp": "^(would reformat (.*))$", + "file": 2, + "message": 1 + } + ] } - ] - } - ] + ] } diff --git a/.github/problemMatcher/clang.json b/.github/problemMatcher/clang.json index b8a1185427..cb137fd4f5 100644 --- a/.github/problemMatcher/clang.json +++ b/.github/problemMatcher/clang.json @@ -1,17 +1,17 @@ { - "problemMatcher": [ - { - "owner": "clang", - "pattern": [ + "problemMatcher": [ { - "regexp": "^(.+):([0-9]+):([0-9]+): (error|warning|note): (.+)$", - "file": 1, - "line": 2, - "column": 3, - "severity": 4, - "message": 5 + "owner": "clang", + "pattern": [ + { + "regexp": "^(.+):([0-9]+):([0-9]+): (error|warning|note): (.+)$", + "file": 1, + "line": 2, + "column": 3, + "severity": 4, + "message": 5 + } + ] } - ] - } - ] + ] } diff --git a/.github/problemMatcher/codespell.json b/.github/problemMatcher/codespell.json index e8f4f8381f..edfc4e4d73 100644 --- a/.github/problemMatcher/codespell.json +++ b/.github/problemMatcher/codespell.json @@ -1,16 +1,16 @@ { - "problemMatcher": [ - { - "owner": "codespell-matcher", - "severity": "warning", - "pattern": [ + "problemMatcher": [ { - "regexp": "^(.+):(\\d+):\\s+(.+)$", - "file": 1, - "line": 2, - "message": 3 + "owner": "codespell-matcher", + "severity": "warning", + "pattern": [ + { + "regexp": "^(.+):(\\d+):\\s+(.+)$", + "file": 1, + "line": 2, + "message": 3 + } + ] } - ] - } - ] + ] } diff --git a/.github/problemMatcher/cpplint.json b/.github/problemMatcher/cpplint.json index 79efedb0f1..4569f85557 100644 --- a/.github/problemMatcher/cpplint.json +++ b/.github/problemMatcher/cpplint.json @@ -1,16 +1,16 @@ { - "problemMatcher": [ - { - "owner": "cpplint", - "severity": "warning", - "pattern": [ + "problemMatcher": [ { - "regexp": "^(.+):([0-9]+): (.+)$", - "file": 1, - "line": 2, - "message": 3 + "owner": "cpplint", + "severity": "warning", + "pattern": [ + { + "regexp": "^(.+):([0-9]+): (.+)$", + "file": 1, + "line": 2, + "message": 3 + } + ] } - ] - } - ] + ] } diff --git a/.github/problemMatcher/grepMatcherWarning.json b/.github/problemMatcher/grepMatcherWarning.json index 49671d4253..0c0e837d9e 100644 --- a/.github/problemMatcher/grepMatcherWarning.json +++ b/.github/problemMatcher/grepMatcherWarning.json @@ -1,16 +1,16 @@ { - "problemMatcher": [ - { - "owner": "grepMatcher-warning", - "severity": "warning", - "pattern": [ + "problemMatcher": [ { - "regexp": "^(.+):([0-9]+):(.+)$", - "file": 1, - "line": 2, - "message": 3 + "owner": "grepMatcher-warning", + "severity": "warning", + "pattern": [ + { + "regexp": "^(.+):([0-9]+):(.+)$", + "file": 1, + "line": 2, + "message": 3 + } + ] } - ] - } - ] + ] } diff --git a/.github/problemMatcher/msvc.json b/.github/problemMatcher/msvc.json index b8802ef820..17d984da6d 100644 --- a/.github/problemMatcher/msvc.json +++ b/.github/problemMatcher/msvc.json @@ -1,11 +1,13 @@ { - "__comment": "Taken from vscode's vs/workbench/contrib/tasks/common/problemMatcher.ts msCompile rule", + "__comment": + "Taken from vscode's vs/workbench/contrib/tasks/common/problemMatcher.ts msCompile rule", "problemMatcher": [ { "owner": "msvc-problem-matcher", "pattern": [ { - "regexp": "^(?:\\s+\\d+\\>)?([^\\s].*)\\((\\d+),?(\\d+)?(?:,\\d+,\\d+)?\\)\\s*:\\s+(error|warning|info)\\s+(\\w{1,2}\\d+)\\s*:\\s*(.*)$", + "regexp": + "^(?:\\s+\\d+\\>)?([^\\s].*)\\((\\d+),?(\\d+)?(?:,\\d+,\\d+)?\\)\\s*:\\s+(error|warning|info)\\s+(\\w{1,2}\\d+)\\s*:\\s*(.*)$", "file": 1, "line": 2, "column": 3, diff --git a/.github/problemMatcher/pylintError.json b/.github/problemMatcher/pylintError.json index 8c5f73fbf5..60bee1542c 100644 --- a/.github/problemMatcher/pylintError.json +++ b/.github/problemMatcher/pylintError.json @@ -1,18 +1,18 @@ { - "problemMatcher": [ - { - "owner": "pylint-error", - "severity": "error", - "pattern": [ + "problemMatcher": [ { - "regexp": "^(.+?):([0-9]+):([0-9]+): ([([E|F][0-9]+): (.*)$", - "file": 1, - "line": 2, - "column": 3, - "code": 4, - "message": 5 + "owner": "pylint-error", + "severity": "error", + "pattern": [ + { + "regexp": "^(.+?):([0-9]+):([0-9]+): ([([E|F][0-9]+): (.*)$", + "file": 1, + "line": 2, + "column": 3, + "code": 4, + "message": 5 + } + ] } - ] - } - ] + ] } diff --git a/.github/problemMatcher/pylintWarning.json b/.github/problemMatcher/pylintWarning.json index 6a453f1d43..b38b4ebcb7 100644 --- a/.github/problemMatcher/pylintWarning.json +++ b/.github/problemMatcher/pylintWarning.json @@ -1,18 +1,18 @@ { - "problemMatcher": [ - { - "owner": "pylint-warning", - "severity": "warning", - "pattern": [ + "problemMatcher": [ { - "regexp": "^(.+?):([0-9]+):([0-9]+): ([^E|F][0-9]+): (.*)$", - "file": 1, - "line": 2, - "column": 3, - "code": 4, - "message": 5 + "owner": "pylint-warning", + "severity": "warning", + "pattern": [ + { + "regexp": "^(.+?):([0-9]+):([0-9]+): ([^E|F][0-9]+): (.*)$", + "file": 1, + "line": 2, + "column": 3, + "code": 4, + "message": 5 + } + ] } - ] - } - ] + ] } diff --git a/.github/workflows/auto-close_stale_issues_and_pull-requests.yml b/.github/workflows/auto-close_stale_issues_and_pull-requests.yml index 9433dbe202..5e853db3c6 100644 --- a/.github/workflows/auto-close_stale_issues_and_pull-requests.yml +++ b/.github/workflows/auto-close_stale_issues_and_pull-requests.yml @@ -130,4 +130,3 @@ jobs: Otherwise, we’ll close this PR automatically in 90 days. If you would like to keep working on this pull request, we advice to rebase it on current main branch, ask feedback from users or maintainers and engage with the community to get it forward. -