[CI] Refactor check-bypass logic in run_tests_with_coverage#4655
Merged
EmmonsCurse merged 3 commits intoPaddlePaddle:developfrom Nov 7, 2025
Merged
[CI] Refactor check-bypass logic in run_tests_with_coverage#4655EmmonsCurse merged 3 commits intoPaddlePaddle:developfrom
EmmonsCurse merged 3 commits intoPaddlePaddle:developfrom
Conversation
|
Thanks for your contribution! |
b74b5d8 to
b9889ab
Compare
b9889ab to
505f8aa
Compare
a568a24 to
d070063
Compare
YuanRisheng
approved these changes
Nov 7, 2025
XieYunshen
approved these changes
Nov 7, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
Improve CI efficiency by updating the check-bypass logic to handle new commits.
Adjusted the logic for the
skip-ci: coveragemechanism to prevent cases where coverage-exempted PRs skip all test verification after code updates.Now, the skip flag only bypasses the coverage check, not the unit test execution.
For PRs with coverage exemption requests, a reminder will be shown to encourage maintainers to review and merge them promptly.
If the code is updated afterward, tests will still be executed to ensure correctness.
Modifications
Updated check-bypass workflow to use commit SHA for skip determination.
Usage or Command
No user-facing changes.
Accuracy Tests
Functionality unaffected, no tests needed.
Checklist
[FDConfig],[APIServer],[Engine],[Scheduler],[PD Disaggregation],[Executor],[Graph Optimization],[Speculative Decoding],[RL],[Models],[Quantization],[Loader],[OP],[KVCache],[DataProcessor],[BugFix],[Docs],[CI],[Optimization],[Feature],[Benchmark],[Others],[XPU],[HPU],[GCU],[DCU],[Iluvatar],[Metax]]pre-commitbefore commit.releasebranch, make sure the PR has been submitted to thedevelopbranch, then cherry-pick it to thereleasebranch with the[Cherry-Pick]PR tag.