Skip to content

Conversation

@Yang-33
Copy link
Contributor

@Yang-33 Yang-33 commented Mar 26, 2025

Changes

This PR specifies minimal permissions on each job to reduce unnecessary access to repository contents and avoid potential security risks. By granting only the required scopes (e.g. read-only for repository contents and write access only for issues to post PR comments), we can protect the supply chain from unintended privilege escalations.

parent: line/line-bot-sdk-nodejs#1202

@github-actions
Copy link

JAVA

You can check generated code in java

Check the diff here

@github-actions
Copy link

PYTHON

You can check generated code in python

Check the diff here

@github-actions
Copy link

GO

You can check generated code in go

Check the diff here

@github-actions
Copy link

PHP

You can check generated code in php

Check the diff here

@github-actions
Copy link

NODEJS

You can check generated code in nodejs

Check the diff here

language: python
github-token: ${{ secrets.GITHUB_TOKEN }}

- name: Update version in linebot/__about__.py
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is an intentional change. line/line-bot-sdk-python#710 forgot deleting these lines in this repo.

- name: Setup PHP
uses: shivammathur/setup-php@9e72090525849c5e82e596468b86eb55e9cc5401 # 2.32.0
with:
distribution: 'temurin'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

distribution: 'temurin' is only for setup-java 😅 some actions may be copied from setup-java, but it's unnecessary.

@Yang-33 Yang-33 requested a review from a team March 26, 2025 12:54
@Yang-33 Yang-33 self-assigned this Mar 26, 2025
Copy link
Contributor

@eucyt eucyt left a comment

Choose a reason for hiding this comment

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

Thank you!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants