Skip to content

fix: Replace pull_request_target with pull_request for improved security#312

Open
Shubham15986 wants to merge 7 commits intobrisbanesocialchess:mainfrom
Shubham15986:fix/workflow-security
Open

fix: Replace pull_request_target with pull_request for improved security#312
Shubham15986 wants to merge 7 commits intobrisbanesocialchess:mainfrom
Shubham15986:fix/workflow-security

Conversation

@Shubham15986
Copy link

  • Replace pull_request_target with pull_request for improved security in labeler workflow
  • Added specific trigger types and path restrictions

Copy link
Contributor

@jbampton jbampton left a comment

Choose a reason for hiding this comment

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

Please remove all path restrictions

@jbampton jbampton self-assigned this Oct 2, 2025
@jbampton jbampton added this to Chess Oct 2, 2025
@jbampton jbampton moved this to In Progress in Chess Oct 2, 2025
Shubham15986 and others added 4 commits October 2, 2025 06:53
- Added explicit branches target
- Improved types formatting
- Fixed spacing and indentation
- Removed unnecessary quotes
- Simplified step name
- Fixed truthy value warning by quoting 'on'
- Maintained proper indentation and structure
- Ensured file passes yamllint checks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

2 participants