Skip to content

[CI] Remove invaild check-bypass step condition in inference workflow#78687

Merged
SigureMo merged 1 commit intoPaddlePaddle:developfrom
ooooo-create:clean_code
Apr 17, 2026
Merged

[CI] Remove invaild check-bypass step condition in inference workflow#78687
SigureMo merged 1 commit intoPaddlePaddle:developfrom
ooooo-create:clean_code

Conversation

@ooooo-create
Copy link
Copy Markdown
Contributor

PR Category

Environment Adaptation

PR Types

Not User Facing

Description

清理无效语句,并没有 steps.check-bypass,只有 needs.check-bypass 且在 job 级就已经判断 check-bypass 了

是否引起精度变化

Copilot AI review requested due to automatic review settings April 16, 2026 01:00
@paddle-bot
Copy link
Copy Markdown

paddle-bot Bot commented Apr 16, 2026

你的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.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 cleans up the GitHub Actions inference workflow by removing an invalid step-level condition that referenced a non-existent steps.check-bypass output, relying instead on the existing job-level needs.check-bypass gating.

Changes:

  • Removed the invalid if: steps.check-bypass.outputs... condition from the “Upload build.tar.gz” step in the inference build job.

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

@ooooo-create ooooo-create changed the title [C] remove invaild check-bypass step condition in inference workflow [CI] remove invaild check-bypass step condition in inference workflow Apr 16, 2026
@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Apr 17, 2026
@SigureMo SigureMo changed the title [CI] remove invaild check-bypass step condition in inference workflow [CI] Remove invaild check-bypass step condition in inference workflow Apr 17, 2026
@SigureMo SigureMo merged commit 8d6943e into PaddlePaddle:develop Apr 17, 2026
283 of 291 checks passed
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: all

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants