[PR #677] [MERGED] code format & enable code format checking in ci #1739

New issue

Closed

opened 2026-10-29 21:29:56 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-29 21:29:56 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/kvcache-ai/Mooncake/pull/677
Author: @doujiang24
Created: 7/26/2025
Status: ✅ Merged
Merged: 8/2/2025
Merged by: @doujiang24

Base: main ← Head: format-check

📝 Commits (1)

e5b3ee2 code format & enable code format checking in ci

📊 Changes

85 files changed (+1917 additions, -1515 deletions)

View changed files

📝 .clang-format (+3 -0)
📝 .github/workflows/ci.yml (+31 -2)
📝 mooncake-integration/transfer_engine/transfer_engine_py.cpp (+71 -59)
📝 mooncake-integration/transfer_engine/transfer_engine_py.h (+25 -24)
📝 mooncake-store/benchmarks/allocator_bench.cpp (+13 -7)
📝 mooncake-store/include/allocation_strategy.h (+8 -5)
📝 mooncake-store/include/allocator.h (+5 -4)
📝 mooncake-store/include/eviction_strategy.h (+8 -8)
📝 mooncake-store/include/file_interface.h (+50 -36)
📝 mooncake-store/include/ha_helper.h (+2 -3)
📝 mooncake-store/include/hf3fs/hf3fs.h (+33 -28)
📝 mooncake-store/include/master_metric_manager.h (+1 -2)
📝 mooncake-store/include/master_service.h (+15 -16)
📝 mooncake-store/include/rpc_service.h (+1 -2)
📝 mooncake-store/include/segment.h (+11 -8)
📝 mooncake-store/include/storage_backend.h (+65 -54)
📝 mooncake-store/include/thread_pool.h (+19 -19)
📝 mooncake-store/include/transfer_task.h (+13 -15)
📝 mooncake-store/include/types.h (+6 -4)
📝 mooncake-store/src/allocator.cpp (+6 -4)

...and 65 more files

📄 Description

Force follow the .clang-format, to reduce code style changes in PRs, make PRs review easier.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/kvcache-ai/Mooncake/pull/677 **Author:** [@doujiang24](https://github.com/doujiang24) **Created:** 7/26/2025 **Status:** ✅ Merged **Merged:** 8/2/2025 **Merged by:** [@doujiang24](https://github.com/doujiang24) **Base:** `main` ← **Head:** `format-check` --- ### 📝 Commits (1) - [`e5b3ee2`](https://github.com/kvcache-ai/Mooncake/commit/e5b3ee2c157be0305dfdbfecbf349dc10e878873) code format & enable code format checking in ci ### 📊 Changes **85 files changed** (+1917 additions, -1515 deletions) <details> <summary>View changed files</summary> 📝 `.clang-format` (+3 -0) 📝 `.github/workflows/ci.yml` (+31 -2) 📝 `mooncake-integration/transfer_engine/transfer_engine_py.cpp` (+71 -59) 📝 `mooncake-integration/transfer_engine/transfer_engine_py.h` (+25 -24) 📝 `mooncake-store/benchmarks/allocator_bench.cpp` (+13 -7) 📝 `mooncake-store/include/allocation_strategy.h` (+8 -5) 📝 `mooncake-store/include/allocator.h` (+5 -4) 📝 `mooncake-store/include/eviction_strategy.h` (+8 -8) 📝 `mooncake-store/include/file_interface.h` (+50 -36) 📝 `mooncake-store/include/ha_helper.h` (+2 -3) 📝 `mooncake-store/include/hf3fs/hf3fs.h` (+33 -28) 📝 `mooncake-store/include/master_metric_manager.h` (+1 -2) 📝 `mooncake-store/include/master_service.h` (+15 -16) 📝 `mooncake-store/include/rpc_service.h` (+1 -2) 📝 `mooncake-store/include/segment.h` (+11 -8) 📝 `mooncake-store/include/storage_backend.h` (+65 -54) 📝 `mooncake-store/include/thread_pool.h` (+19 -19) 📝 `mooncake-store/include/transfer_task.h` (+13 -15) 📝 `mooncake-store/include/types.h` (+6 -4) 📝 `mooncake-store/src/allocator.cpp` (+6 -4) _...and 65 more files_ </details> ### 📄 Description Force follow the .clang-format, to reduce code style changes in PRs, make PRs review easier. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-29 21:29:56 +00:00