Skip to content

Enable use of clang-format - #740

Open
Chuck Walbourn (walbourn) wants to merge 11 commits into
mainfrom
clangformat
Open

Enable use of clang-format#740
Chuck Walbourn (walbourn) wants to merge 11 commits into
mainfrom
clangformat

Conversation

@walbourn

@walbourn Chuck Walbourn (walbourn) commented Sep 7, 2026

Copy link
Copy Markdown
Collaborator
  • Enable clang-format for GitHub Super Linter

  • Added clang format script: powershell -File build/Format.ps1

Local reformat has to exactly match the SuperLinter version. winget install --id=LLVM.LLVM --version 21.1.2 installs the matching version. You use powershell -File build/Format.ps1 -LLVM to use that version.

  • Added GHA workarounds for build failures due to ICE in openjph via vpkg for x86-windows.

@walbourn Chuck Walbourn (walbourn) added maintenance github_actions Pull requests that update GitHub Actions code labels Sep 7, 2026
@walbourn

Chuck Walbourn (walbourn) commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator Author

The x86 checks are failing due to a MSVC compiler bug with openjph via vcpkg. Otherwise everything builds fine. Since a fix is taking a while, I've removed VS 2022 x86 combos that use VCPKG.

@walbourn

Copy link
Copy Markdown
Collaborator Author

NOTE that DirectXTex.inl is not scanned by Super-Linter currently. This has been reported via super-linter/super-linter#8110

Comment thread .github/linters/.clang-format
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code maintenance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant