Skip to content

[3.13] gh-127785: Limit check labels github action permission (GH-130596)#130623

Merged
hugovk merged 4 commits into
python:3.13from
shenxianpeng:backport-5ba69e7-3.13
Feb 27, 2025
Merged

[3.13] gh-127785: Limit check labels github action permission (GH-130596)#130623
hugovk merged 4 commits into
python:3.13from
shenxianpeng:backport-5ba69e7-3.13

Conversation

@shenxianpeng

@shenxianpeng shenxianpeng commented Feb 27, 2025

Copy link
Copy Markdown
Contributor

…flow (pythonGH-130596)

(cherry picked from commit 5ba69e7)

Co-authored-by: shenxianpeng <xianpeng.shen@gmail.com>
@shenxianpeng shenxianpeng changed the title [3.12] gh-127785: Limit check labels github action permission (GH-130596) [3.13] gh-127785: Limit check labels github action permission (GH-130596) Feb 27, 2025
@hugovk

hugovk commented Feb 27, 2025

Copy link
Copy Markdown
Member

Thank you for the backport. Too much has changed in this PR, please could you limit it to the permissions changes? Same applies to #130625.

@shenxianpeng

Copy link
Copy Markdown
Contributor Author

I have updated the same applies to #130625

@hugovk

hugovk commented Feb 27, 2025

Copy link
Copy Markdown
Member

This still has a lot of unrelated changes. Please only change:

    permissions:
      issues: write
      pull-requests: write

to:

    permissions:
      pull-requests: read

@hugovk hugovk enabled auto-merge (squash) February 27, 2025 14:33
@hugovk hugovk disabled auto-merge February 27, 2025 14:34

@hugovk hugovk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@hugovk hugovk enabled auto-merge (squash) February 27, 2025 14:35
@hugovk hugovk merged commit 676ceca into python:3.13 Feb 27, 2025
@shenxianpeng shenxianpeng deleted the backport-5ba69e7-3.13 branch February 27, 2025 15:03
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