Skip to content

Fix codeformat script for PEP 668 and apply code formatting.#3299

Merged
shlzxjp merged 1 commit intomainfrom
bugfix/hparty_codeformat
Mar 3, 2026
Merged

Fix codeformat script for PEP 668 and apply code formatting.#3299
shlzxjp merged 1 commit intomainfrom
bugfix/hparty_codeformat

Conversation

@Hparty
Copy link
Collaborator

@Hparty Hparty commented Mar 2, 2026

修复 codeformat.sh 在新版 Python (3.12+) 环境下安装 clang-format 失败的问题,改用 pipx 安装。

主要改动:

  • pip3 install 改为 pipx install,避免 PEP 668 限制
  • 设置 PATH 优先使用 pipx 安装的 clang-format 版本
  • 应用 clang-format 14.0.0 格式化修正

参考:#3118

@Hparty Hparty force-pushed the bugfix/hparty_codeformat branch from da0dcd1 to 24d8f91 Compare March 3, 2026 02:39
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.84%. Comparing base (ce1c42f) to head (24d8f91).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3299      +/-   ##
==========================================
+ Coverage   74.76%   74.84%   +0.08%     
==========================================
  Files         501      501              
  Lines       32765    32765              
  Branches    10701    10701              
==========================================
+ Hits        24496    24523      +27     
+ Misses       6059     6052       -7     
+ Partials     2210     2190      -20     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@shlzxjp shlzxjp merged commit 3ebe074 into main Mar 3, 2026
8 checks passed
@shlzxjp shlzxjp deleted the bugfix/hparty_codeformat branch March 3, 2026 06:09
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.

3 participants