Skip to content

chore(ci): 为工作流声明最小权限 (contents: read)#96

Merged
Awuqing merged 1 commit into
mainfrom
chore/ci-permissions
May 31, 2026
Merged

chore(ci): 为工作流声明最小权限 (contents: read)#96
Awuqing merged 1 commit into
mainfrom
chore/ci-permissions

Conversation

@Awuqing

@Awuqing Awuqing commented May 31, 2026

Copy link
Copy Markdown
Owner

背景

修复仓库 CodeQL 的两处 actions/missing-workflow-permissions 告警(.github/workflows/ci.yml:11:31):工作流未显式声明 permissionsGITHUB_TOKEN 默认携带可写权限。

改动

ci.yml 顶层声明最小权限 permissions: contents: read——构建/测试 job 仅需检出与读取仓库内容,无需写权限。

影响

纯 CI 配置加固,不影响构建/测试逻辑;同时清掉本仓库剩余的两个未决 CodeQL 告警(其余 6 个已按误报/设计预期完成三态化)。

修复 CodeQL actions/missing-workflow-permissions 告警:ci.yml 未显式声明
GITHUB_TOKEN 权限,默认可写。构建/测试仅需读取仓库内容,故收敛为 contents: read。
@Awuqing Awuqing merged commit bf0e91d into main May 31, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant