Skip to content

fix: exclude spine 4.2 third-party files from clang-format - #273

Merged
star-e merged 4 commits into
v4.0.0from
copilot/fix-clang-format-job
Aug 15, 2026
Merged

fix: exclude spine 4.2 third-party files from clang-format#273
star-e merged 4 commits into
v4.0.0from
copilot/fix-clang-format-job

Conversation

Copilot AI commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

The Clang-Format CI job was reformatting 587 third-party spine 4.2 files on every run, causing the automated push to fail with fatal: shallow file has changed since we read it.

Changes

  • native/.clang-format-ignore — added ./cocos/editor-support/spine/4.2/* alongside existing third-party exclusions (bindings/auto, tetgen)
  • .github/workflows/native-clang-format.yml — added ./native/cocos/editor-support/spine/4.2 to the DoozyX/clang-format-lint-action exclude list so the action itself skips those files

测试方法

不需要测试,用于cocos-robot提交clang format后的代码。

Co-authored-by: star-e <13654656+star-e@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job Clang-Format fix: exclude spine 4.2 third-party files from clang-format Aug 15, 2026
Copilot AI requested a review from star-e August 15, 2026 07:42
@star-e
star-e marked this pull request as ready for review August 15, 2026 07:50
@github-actions

Copy link
Copy Markdown

Code Size Check Report

Wechat (WASM) Before After Diff
2D Empty (legacy pipeline) 1015084 bytes 1015084 bytes ✅ 0 bytes
2D All (legacy pipeline) 2682815 bytes 2682815 bytes ✅ 0 bytes
2D All (new pipeline) 2774788 bytes 2774788 bytes ✅ 0 bytes
(2D + 3D) All 10032207 bytes 10032207 bytes ✅ 0 bytes
Web (WASM + ASMJS) Before After Diff
(2D + 3D) All 16868625 bytes 16868625 bytes ✅ 0 bytes

Interface Check Report

This pull request does not change any public interfaces !

@star-e
star-e merged commit 01983b7 into v4.0.0 Aug 15, 2026
37 checks passed
@star-e
star-e deleted the copilot/fix-clang-format-job branch August 15, 2026 10:34
star-e added a commit that referenced this pull request Aug 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants