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

run clang-format in CI #1692

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
4 changes: 3 additions & 1 deletion .clang-format
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ BasedOnStyle: LLVM
BreakConstructorInitializersBeforeComma: true
ConstructorInitializerAllOnOneLineOrOnePerLine: true
Cpp11BracedListStyle: true
Standard: Cpp11
Standard: c++20
#SpaceBeforeParens: ControlStatements
SpaceAfterControlStatementKeyword: true
PointerBindsToType: true
Expand All @@ -12,4 +12,6 @@ UseTab: Never
ColumnLimit: 100
NamespaceIndentation: Inner
AlignConsecutiveAssignments: true
SortIncludes: Never
ReflowComments: false
...
20 changes: 10 additions & 10 deletions .devcontainer/devcontainer.json
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
{
"image": "ghcr.io/eic/jug_prod:master-nightly",
"postCreateCommand": "./.devcontainer/postCreateCommand.sh",
"customizations": {
"vscode": {
"extensions": [
"ms-vscode.cmake-tools",
"ms-vscode.cpptools",
"ms-vscode.cpptools-extension-pack"
]
}
"image": "ghcr.io/eic/jug_prod:master-nightly",
"postCreateCommand": "./.devcontainer/postCreateCommand.sh",
"customizations": {
"vscode": {
"extensions": [
"ms-vscode.cmake-tools",
"ms-vscode.cpptools",
"ms-vscode.cpptools-extension-pack"
]
}
}
}
22 changes: 11 additions & 11 deletions .github/eicrecon.json
Original file line number Diff line number Diff line change
@@ -1,15 +1,15 @@
{
"problemMatcher": [
"problemMatcher": [
{
"owner": "eicrecon",
"pattern": [
{
"owner": "eicrecon",
"pattern": [
{
"regexp": "^\\[(.*)\\]\\s\\[(error)\\]\\s*(.*)$",
"file": 1,
"severity": 2,
"message": 3
}
]
"regexp": "^\\[(.*)\\]\\s\\[(error)\\]\\s*(.*)$",
"file": 1,
"severity": 2,
"message": 3
}
]
]
}
]
}
30 changes: 15 additions & 15 deletions .github/ubsan.json
Original file line number Diff line number Diff line change
@@ -1,18 +1,18 @@
{
"problemMatcher": [
"problemMatcher": [
{
"owner": "ubsan",
"pattern": [
{
"owner": "ubsan",
"pattern": [
{
"regexp": "^.*/(src/.+):(\\d+):(\\d+):\\sruntime\\s(error):\\s*(.*)$",
"file": 1,
"fromPath": 1,
"line": 2,
"column": 3,
"severity": 4,
"message": 5
},
]
}
]
"regexp": "^.*/(src/.+):(\\d+):(\\d+):\\sruntime\\s(error):\\s*(.*)$",
"file": 1,
"fromPath": 1,
"line": 2,
"column": 3,
"severity": 4,
"message": 5
},
]
}
]
}
12 changes: 5 additions & 7 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
@@ -1,15 +1,18 @@
ci:
skip: [clang-format]
repos:
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v5.0.0
hooks:
- id: check-yaml
- id: end-of-file-fixer
- id: trailing-whitespace
- repo: https://github.com/codespell-project/codespell
rev: v2.3.0
hooks:
- id: codespell
- repo: https://github.com/pre-commit/mirrors-clang-format
rev: v19.1.4
hooks:
- id: clang-format
- repo: https://github.com/Lucas-C/pre-commit-hooks
rev: v1.5.5
hooks:
Expand All @@ -18,11 +21,6 @@ repos:
- id: forbid-tabs
- id: remove-tabs
args: [--whitespaces-count, '8']
- repo: https://github.com/pocc/pre-commit-hooks
rev: v1.3.5
hooks:
- id: clang-format
args: [-i]
- repo: https://github.com/cheshirekow/cmake-format-precommit
rev: v0.6.13
hooks:
Expand Down
5,999 changes: 5,142 additions & 857 deletions docs/arches_flags.json

Large diffs are not rendered by default.

Loading
Loading