Skip to content

fix(ci): make clang-format check merge-race safe - #487

Merged
KemengHuang merged 1 commit into
mainfrom
refactor-main
Aug 31, 2026
Merged

fix(ci): make clang-format check merge-race safe#487
KemengHuang merged 1 commit into
mainfrom
refactor-main

Conversation

@KemengHuang

Copy link
Copy Markdown
Member

Summary

  • compare clang-format inputs using the immutable pull-request base/head SHAs
  • remove the moving shallow fetch that can lose the merge base when a PR merges while the job is running
  • add a repository contract for the workflow invariant

Validation

  • Repository Contracts: 32/32
  • local clang-format-18 check for the merged source change: passed

@KemengHuang
KemengHuang merged commit b6f2b00 into main Aug 31, 2026
12 of 21 checks passed
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.

1 participant