Skip to content

add presion approver#4772

Merged
zjjlivein merged 1 commit into
PaddlePaddle:developfrom
zjjlivein:add_update_precison_approve
Jul 16, 2026
Merged

add presion approver#4772
zjjlivein merged 1 commit into
PaddlePaddle:developfrom
zjjlivein:add_update_precison_approve

Conversation

@zjjlivein

Copy link
Copy Markdown
Collaborator

Before submitting

  • Lint code. If there are lint issues, please format the code first.
# Install and register `pre-commit` in the project folder
pip install pre-commit && pre-commit install

# Process previous code files separately
pre-commit run --file XXXX.py
  • Add test cases into tests folder. If there are codecov issues, please add tests cases first.

PR types

PR changes

Description

@paddle-bot

paddle-bot Bot commented Jul 16, 2026

Copy link
Copy Markdown

Thanks for your contribution!

@zjjlivein zjjlivein merged commit 0629ad3 into PaddlePaddle:develop Jul 16, 2026
11 checks passed

@risemeup1111 risemeup1111 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

已复查本次变更,代码层面未发现需要阻塞合入的问题。查询时仍有部分 CI job 未完成,最终状态请以全部 CI 完成后的结果为准。

P3 优先级:P3 非行级:PR 标题/描述不在 diff 中,无法挂到行级评论。PR 标题存在拼写错误(presion),描述也还保留模板占位。这个变更是在放宽 precision 审批白名单,建议补充清晰标题和描述,例如 add precision approver zjjlivein,并说明新增审批人的权限依据、影响范围和测试/CI 情况,方便后续追溯。

处理要求:请针对该评论进行回复(同意并已修改请回复 Done,不同意请说明理由)。

Powered by Nyanpasu with gpt-5.5 xhigh, please check the suggestions carefully.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants