Skip to content

Add code formatting CI with clang-format and black#5

Merged
erwei-xilinx merged 1 commit into
amd:mainfrom
erwei-xilinx:add-formatting-ci
Feb 24, 2026
Merged

Add code formatting CI with clang-format and black#5
erwei-xilinx merged 1 commit into
amd:mainfrom
erwei-xilinx:add-formatting-ci

Conversation

@erwei-xilinx

Copy link
Copy Markdown
Collaborator

Summary

  • Add a GitHub Actions workflow (.github/workflows/format.yml) that checks C/C++ formatting via clang-format (LLVM style) and Python formatting via black on PRs and pushes to main
  • Add .clang-format config with LLVM base style
  • Add [tool.black] config to pyproject.toml with exclusions for third_party/ and build/
  • Uses reviewdog/action-suggester to post inline PR suggestions for formatting fixes

Test plan

  • Verify the Code Formatting Check workflow triggers on this PR
  • Confirm clang-format check runs against changed C/C++ files
  • Confirm black check runs against Python files (excluding third_party/)
  • Verify reviewdog posts inline suggestions if formatting issues are found

🤖 Generated with Claude Code

Add a GitHub Actions workflow that checks C/C++ formatting via
clang-format (LLVM style) and Python formatting via black on PRs
and pushes to main. Uses reviewdog to post inline PR suggestions.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@erwei-xilinx erwei-xilinx merged commit 4237ea3 into amd:main Feb 24, 2026
7 of 8 checks passed
@erwei-xilinx erwei-xilinx deleted the add-formatting-ci branch February 24, 2026 20:49
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