Skip to content

Refactor config structure and add CI workflow configuration#23

Open
FrozenLemonTee wants to merge 6 commits intomainfrom
add-ci-and-update-config
Open

Refactor config structure and add CI workflow configuration#23
FrozenLemonTee wants to merge 6 commits intomainfrom
add-ci-and-update-config

Conversation

@FrozenLemonTee
Copy link
Member

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR moves configuration files (.clang-format, .clang-tidy, .editorconfig, .gitattributes) into a config/ directory, adds a new GitHub Actions CI workflow for clang-format checking/auto-fixing, and updates documentation references to reflect the new file paths.

Changes:

  • Moved .clang-format, .clang-tidy, .editorconfig, and .gitattributes into config/ and added a new CI workflow at config/.github/workflows/code-format.yml.
  • Updated documentation links in README.md, clang-format.md, and clang-tidy.md to point to the new config/ paths.
  • Added a new "CI/CD 工作流配置" section (§4.2) in README.md describing the workflow capabilities.

Reviewed changes

Copilot reviewed 4 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
config/.github/workflows/code-format.yml New CI workflow for clang-format check and auto-fix
config/.gitattributes Git attributes config moved into config/
config/.editorconfig EditorConfig moved into config/
config/.clang-tidy Clang-tidy config moved into config/
config/.clang-format Clang-format config moved into config/
clang-tidy.md Updated reference path to .clang-tidy
clang-format.md Updated reference path to .clang-format
README.md Updated links to config files and added CI/CD section

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR moves configuration files (.clang-format, .clang-tidy, .editorconfig, .gitattributes) into a config/ subdirectory, adds a new GitHub Actions CI workflow for automated clang-format checking/fixing, and updates documentation references accordingly.

Changes:

  • Adds new config files (.clang-format, .clang-tidy, .editorconfig, .gitattributes) under config/ directory with comprehensive formatting, linting, editor, and Git settings.
  • Adds a GitHub Actions workflow (code-format.yml) that checks formatting on PRs/pushes and optionally auto-fixes via workflow_dispatch.
  • Updates README.md, clang-format.md, and clang-tidy.md to reference the new config/ file paths and adds CI/CD documentation.

Reviewed changes

Copilot reviewed 4 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
config/.github/workflows/code-format.yml New CI workflow for clang-format check and auto-fix
config/.gitattributes Git line-ending and binary file configuration
config/.editorconfig Editor behavior configuration for multiple file types
config/.clang-tidy Static analysis rules for clang-tidy
config/.clang-format C++ code formatting rules
clang-tidy.md Updated config file path reference
clang-format.md Updated config file path reference
README.md Updated links to config files and added CI/CD documentation section

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

FrozenLemonTee and others added 2 commits March 13, 2026 11:41
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
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