Skip to content

ci: Review PR #496 - Reduce usage of token with elevated permissions#514

Open
dellagustin-sap wants to merge 1 commit into
use_app_securityfrom
suggestion#496__PR-Review-min-permissions
Open

ci: Review PR #496 - Reduce usage of token with elevated permissions#514
dellagustin-sap wants to merge 1 commit into
use_app_securityfrom
suggestion#496__PR-Review-min-permissions

Conversation

@dellagustin-sap
Copy link
Copy Markdown

This commit refactors the release.yml workflow to to reduce the usage of the token generated with the GitHub app.

actions/checkout is now using the default token and not persisting credentials, as they are not needed (e.g., Upload VSIX to releases receives the token through the environment.

The token is now generated just before it is needed.

This commit refactors the `release.yml` workflow to to reduce
the usage of the token generated with the GitHub app.

actions/checkout is now using the default token and not persisting
credentials, as they are not needed (e.g., _Upload VSIX to releases_
receives the token through the environment.

The token is now generated just before it is needed.
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.

1 participant