Skip to content

Commit

Permalink
build: upgrade clang-format to v18
Browse files Browse the repository at this point in the history
PR-URL: #53957
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
RedYetiDev authored and RafaelGSS committed Sep 17, 2024
1 parent 8385958 commit 9a07527
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 311 deletions.
4 changes: 2 additions & 2 deletions .clang-format
Original file line number Diff line number Diff line change
Expand Up @@ -9,11 +9,11 @@ AlignEscapedNewlines: Right
AlignOperands: true
AlignTrailingComments: true
AllowAllParametersOfDeclarationOnNextLine: true
AllowShortBlocksOnASingleLine: false
AllowShortBlocksOnASingleLine: Never
AllowShortCaseLabelsOnASingleLine: false
AllowShortFunctionsOnASingleLine: Inline
AllowShortIfStatementsOnASingleLine: true
AllowShortLoopsOnASingleLine: true
AllowShortLoopsOnASingleLine: false
AlwaysBreakAfterDefinitionReturnType: None
AlwaysBreakAfterReturnType: None
AlwaysBreakBeforeMultilineStrings: false
Expand Down
317 changes: 12 additions & 305 deletions tools/clang-format/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 1 addition & 4 deletions tools/clang-format/package.json
Original file line number Diff line number Diff line change
@@ -1,9 +1,6 @@
{
"name": "node-core-clang-format",
"version": "1.0.0",
"description": "Formatting C++ files for Node.js core",
"license": "MIT",
"dependencies": {
"clang-format": "^1.8.0"
"@wasm-fmt/clang-format": "^18.1.8"
}
}

0 comments on commit 9a07527

Please sign in to comment.