Skip to content
Snippets Groups Projects

Fix clang-format off directive for older LLVM versions

Merged Frederik Hennig requested to merge fhennig/clang-format-off into master

The possibility to add an explanation after the clang-format off directive was only added in LLVM 17. Move the explanation to a separate comment to make sure older LLVM versions format correctly.

Merge request reports

Pipeline passed for 1797ecab on fhennig/clang-format-off

Test coverage 89.00% (0.00%) from 2 jobs
Approval is optional

Merged by Christoph AltChristoph Alt 2 days ago (Apr 4, 2025 7:14am UTC)

Merge details

  • Changes merged into master with a481ccf2 (commits were squashed).
  • Deleted the source branch.

Pipeline passed for a481ccf2 on master

Test coverage 89.00% (0.00%) from 2 jobs

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply