Skip to content

chore: cargo fmtの適用とCI(build/clippy/fmt)の追加#9

Merged
suzuki-toshihir0 merged 2 commits into
mainfrom
chore/fmt-and-ci
Feb 26, 2026
Merged

chore: cargo fmtの適用とCI(build/clippy/fmt)の追加#9
suzuki-toshihir0 merged 2 commits into
mainfrom
chore/fmt-and-ci

Conversation

@suzuki-toshihir0

Copy link
Copy Markdown
Owner

Summary

  • cargo fmt の適用(コードスタイルの統一)
  • CI に以下のジョブを追加
    • Build: cargo build
    • Clippy: cargo clippy -- -D warnings
    • Format: cargo fmt --check

Test plan

  • CI の各ジョブ(test/build/clippy/fmt)が全て通ることを確認

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings February 26, 2026 12:42
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown

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 applies automatic code formatting via cargo fmt to ensure consistent code style across the codebase and enhances the CI pipeline by adding three new jobs: Build, Clippy (linting), and Format checking. These additions strengthen the quality assurance process by catching build failures, lint warnings, and formatting inconsistencies early.

Changes:

  • Applied cargo fmt to format Rust source files according to standard conventions
  • Added Build job to verify the project compiles successfully
  • Added Clippy job to enforce lint-free code with warnings treated as errors
  • Added Format job to ensure all code adheres to standard Rust formatting

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

File Description
tests/integration_test.rs Reformatted multi-line assert macro for better readability
src/formatter.rs Split long method chain across multiple lines per Rust formatting conventions
src/config.rs Reformatted method chains and removed trailing whitespace at end of file
.github/workflows/ci.yml Added three new CI jobs (build, clippy, fmt) to enhance quality checks

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

@suzuki-toshihir0 suzuki-toshihir0 enabled auto-merge (squash) February 26, 2026 12:46
@suzuki-toshihir0 suzuki-toshihir0 merged commit 88aa725 into main Feb 26, 2026
4 checks passed
@suzuki-toshihir0 suzuki-toshihir0 deleted the chore/fmt-and-ci branch February 26, 2026 12:51
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