Skip to content

fix(codeql): Workflow does not contain permissions - #814

Merged
lizhengfeng101 merged 4 commits into
alibaba:mainfrom
wu21-web:patch-1
Aug 10, 2026
Merged

fix(codeql): Workflow does not contain permissions#814
lizhengfeng101 merged 4 commits into
alibaba:mainfrom
wu21-web:patch-1

Conversation

@wu21-web

@wu21-web wu21-web commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Description

Screenshot 2026-08-09 at 9 38 17 PM

This appeared recently, just sets missing permissions of workflow introduced in #596 to enhance workflow security

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • CI / Build / Tooling

How Has This Been Tested?

This workflow change does not need any tests.

Checklist

  • New and existing unit tests pass locally with my changes
  • I have signed the CLA

Related Issues

none yet

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

OpenCodeReview: Review complete: 0 finding(s) across 1 selected item(s).

Comment thread .github/workflows/translation-sync.yml Outdated
Co-authored-by: Lei Zhang <61303077+stay-foolish-forever@users.noreply.github.com>

@lizhengfeng101 lizhengfeng101 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@lizhengfeng101
lizhengfeng101 merged commit 372d3dd into alibaba:main Aug 10, 2026
11 checks passed
@wu21-web
wu21-web deleted the patch-1 branch August 10, 2026 11:16
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.

4 participants