Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Feb 27, 2025

Conversation

shenxianpeng
Copy link
Contributor

@shenxianpeng shenxianpeng commented Feb 27, 2025

@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
Copy link
Member

hugovk commented Feb 27, 2025

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
Contributor Author

I have updated the same applies to #130625

@hugovk
Copy link
Member

hugovk commented Feb 27, 2025

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
Copy link
Member

@hugovk hugovk left a comment

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
38 checks passed
@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