Skip to content

feat!: Remove contents: read workflow permission#105

Merged
chazypro merged 2 commits intomainfrom
ref-ghtoken-perms
Aug 31, 2024
Merged

feat!: Remove contents: read workflow permission#105
chazypro merged 2 commits intomainfrom
ref-ghtoken-perms

Conversation

@chris3ware
Copy link
Copy Markdown
Member

@chris3ware chris3ware commented Aug 31, 2024

For consistency with other workflows, the permissions scope for GITHUB_TOKEN has been set to {} at the workflow level and the relevant permissions have been added at the job level.

Calling workflows will need to update their permissions accordingly

The permissions removed have been added at the job level.
For code consistency with other workflows
@chris3ware chris3ware requested a review from a team as a code owner August 31, 2024 06:19
@chazypro chazypro merged commit 2b772d3 into main Aug 31, 2024
@chazypro chazypro deleted the ref-ghtoken-perms branch August 31, 2024 08:45
3ware-release Bot pushed a commit that referenced this pull request Aug 31, 2024
## [4.0.0](v3.2.0...v4.0.0) (2024-08-31)

### ⚠ BREAKING CHANGES

* Remove `contents: read` workflow permission (#105)

### Features

* Remove `contents: read` workflow permission ([#105](#105)) ([2b772d3](2b772d3))
@3ware-release
Copy link
Copy Markdown
Contributor

3ware-release Bot commented Aug 31, 2024

This PR is included in version 4.0.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants