Skip to content

Update permissions to allow release asset attachment during release deploys#364

Merged
jeffpaul merged 4 commits into
developfrom
fix/deploy-asset
Sep 8, 2025
Merged

Update permissions to allow release asset attachment during release deploys#364
jeffpaul merged 4 commits into
developfrom
fix/deploy-asset

Conversation

@jeffpaul
Copy link
Copy Markdown
Member

@jeffpaul jeffpaul commented Aug 7, 2025

Description of the Change

This pull request makes a small update to the deployment workflow permissions for WordPress.org. The permissions are expanded to allow writing to both contents and packages, which may be necessary for publishing or updating packages during deployment.

How to test the Change

Changelog Entry

Developer - Fix issue with attaching release assets during release deploy action.

Credits

Props @jeffpaul.

Checklist:

@jeffpaul jeffpaul added this to the 7.6.0 milestone Aug 7, 2025
@jeffpaul jeffpaul requested a review from dkotter August 7, 2025 14:22
@jeffpaul jeffpaul self-assigned this Aug 7, 2025
@jeffpaul jeffpaul moved this to Code Review in Open Source Practice Aug 7, 2025
@github-actions github-actions Bot added the needs:code-review This requires code review. label Aug 7, 2025
dkotter
dkotter previously approved these changes Aug 7, 2025
@github-project-automation github-project-automation Bot moved this from Code Review to QA Testing in Open Source Practice Aug 7, 2025
@jeffpaul
Copy link
Copy Markdown
Member Author

Reduced packages scope to read, waiting on checks to pass before merging in

@jeffpaul jeffpaul merged commit c71d4da into develop Sep 8, 2025
13 checks passed
@jeffpaul jeffpaul deleted the fix/deploy-asset branch September 8, 2025 20:22
@github-project-automation github-project-automation Bot moved this from QA Testing to Done in Open Source Practice Sep 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs:code-review This requires code review.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants