Skip to content

chore: update pr-check template for security - #19

Merged
GuangmingLuo merged 4 commits into
mainfrom
fix/ci
Jun 27, 2025
Merged

chore: update pr-check template for security#19
GuangmingLuo merged 4 commits into
mainfrom
fix/ci

Conversation

@GuangmingLuo

@GuangmingLuo GuangmingLuo commented Jun 27, 2025

Copy link
Copy Markdown
Member

Change-Id: Ic99cc4a2bd6aec38cb6746ab56cb1fe3c15dc8ec

What type of PR is this?

Check the PR title.

  • This PR title match the format: <type>(optional scope): <description>
  • The description of this PR title is user-oriented and clear enough for others to understand.
  • Attach the PR updating the user documentation if the current PR requires user awareness at the usage level. User docs repo

(Optional) Translate the PR title into Chinese.

(Optional) More detailed description for this PR(en: English/zh: Chinese).

en:
zh(optional):

  1. 对于 pull_request 添加CI触发条件;对于 push 无需额外配置(默认写权限才能push)
  2. 每个具体的 job 检查前置权限检查结果
  3. pull-requests 权限默认声明为 read,具体的job可以按需显示开启 write

(Optional) Which issue(s) this PR fixes:

(optional) The PR that updates user documentation:

Change-Id: Ic99cc4a2bd6aec38cb6746ab56cb1fe3c15dc8ec
Comment thread .github/workflows/pr-check.yml Fixed
Comment thread .github/workflows/pr-check.yml Fixed
Comment thread .github/workflows/pr-check.yml Fixed
Change-Id: Icdb5be7c1571b56ac0d189ed5c08d4689a9d7026
Change-Id: If6589424100a446a5813be11d0cdff69cbe5b677
Change-Id: I4c4681483e706be68f4d1aa264a1225f3697ff31

@Xander-ByteDance Xander-ByteDance left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants