Skip to content

Conversation

@gouzil
Copy link
Member

@gouzil gouzil commented Aug 18, 2025

PR Category

User Experience

PR Types

Not User Facing

Description

使用性能更好,格式化效果更佳的 ruff format 替代 black 作为新的 formatter,由于我们很早就已经集成了 ruff lint,因此只是减少依赖而不会增加依赖

@gouzil gouzil requested a review from SigureMo as a code owner August 18, 2025 13:48
@paddle-bot
Copy link

paddle-bot bot commented Aug 18, 2025

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot paddle-bot bot added the contributor External developers label Aug 18, 2025
@SigureMo SigureMo added the HappyOpenSource 快乐开源活动issue与PR label Aug 18, 2025
SigureMo
SigureMo previously approved these changes Aug 18, 2025
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 66.75532% with 125 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (develop@a5d987e). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...n/paddle/distributed/auto_parallel/static/utils.py 62.50% 12 Missing ⚠️
...ted/auto_parallel/intermediate/context_parallel.py 0.00% 11 Missing ⚠️
...ted/auto_parallel/static/tuner/rule_based_tuner.py 50.00% 10 Missing ⚠️
...allel/static/operators/dist_update_loss_scaling.py 0.00% 9 Missing ⚠️
...uted/auto_parallel/static/operators/dist_matmul.py 55.55% 8 Missing ⚠️
...ddle/distributed/auto_parallel/pipelining/stage.py 61.11% 7 Missing ⚠️
...paddle/distributed/auto_parallel/high_level_api.py 0.00% 6 Missing ⚠️
...dle/distributed/auto_parallel/static/completion.py 58.33% 5 Missing ⚠️
.../paddle/distributed/auto_parallel/static/engine.py 73.68% 5 Missing ⚠️
...tributed/auto_parallel/intermediate/parallelize.py 33.33% 4 Missing ⚠️
... and 33 more

❌ Your patch status has failed because the patch coverage (66.75%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop   #74713   +/-   ##
==========================================
  Coverage           ?   66.75%           
==========================================
  Files              ?       72           
  Lines              ?      376           
  Branches           ?        0           
==========================================
  Hits               ?      251           
  Misses             ?      125           
  Partials           ?        0           

☔ 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.

@SigureMo SigureMo merged commit 1995c8f into PaddlePaddle:develop Aug 19, 2025
117 of 122 checks passed
@SigureMo SigureMo deleted the ruff/fmt/part-26 branch August 19, 2025 06:06
Luckycheng222 pushed a commit to Luckycheng222/Paddle that referenced this pull request Aug 25, 2025
…74713)

---------

Co-authored-by: SigureMo <sigure.qaq@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor External developers HappyOpenSource 快乐开源活动issue与PR skip-ci: approval

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants