X Tutup
Skip to content

CI: Bump clang-format and clang-tidy versions#110968

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
Repiteo:ci/bump-clang-format-tidy
Dec 15, 2025
Merged

CI: Bump clang-format and clang-tidy versions#110968
Repiteo merged 1 commit intogodotengine:masterfrom
Repiteo:ci/bump-clang-format-tidy

Conversation

@Repiteo
Copy link
Contributor

@Repiteo Repiteo commented Sep 27, 2025

  • clang-format hook: 20.1.0 → 21.1.7
  • clang-tidy hook: 20.1.0 → 21.1.6
  • .clang-format file: 17.0.6 → 18.1.8 (kept 3 versions behind latest for compatibility)

The three script files changed were removing the trailing comma from bin-packed arrays, as that's meant to enforce a one-per-line syntax but earlier versions didn't properly handle comments on the same line. The alternatives were either leaving the trailing commas but causing the packed arguments to be multiline, or wrapping these cases in /* clang-format off/on */; both resulted in more version control noise

@Repiteo Repiteo added this to the 4.x milestone Sep 27, 2025
@Repiteo Repiteo requested a review from a team September 27, 2025 15:37
@Repiteo Repiteo requested a review from a team as a code owner September 27, 2025 15:37
@Repiteo Repiteo requested a review from a team as a code owner September 27, 2025 15:37
@Repiteo Repiteo removed request for a team September 27, 2025 15:37
@Repiteo Repiteo force-pushed the ci/bump-clang-format-tidy branch from 95bc429 to 28a2748 Compare September 27, 2025 15:37
@Repiteo Repiteo force-pushed the ci/bump-clang-format-tidy branch 2 times, most recently from 2cf738c to 47341ae Compare December 2, 2025 20:05
• clang-format: 20.1.0 → 21.1.7
• clang-tidy: 20.1.0 → 21.1.6
• `.clang-format`: 17.0.6 → 18.1.8 (keep 3 versions behind latest for compatibility)
@Repiteo Repiteo modified the milestones: 4.x, 4.6 Dec 13, 2025
@Repiteo Repiteo merged commit 8aa4a52 into godotengine:master Dec 15, 2025
20 checks passed
@Repiteo Repiteo deleted the ci/bump-clang-format-tidy branch December 15, 2025 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

X Tutup