-
Notifications
You must be signed in to change notification settings - Fork 117
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CI: Update checkout settings #3524
Conversation
Change-type: patch
Website deployed to CF Pages, 👀 preview link https://50943deb.balena-os.pages.dev |
@@ -55,6 +55,7 @@ jobs: | |||
# The default GITHUB_TOKEN does not have workflow scope | |||
# This is needed to push a new branch with workflow files | |||
token: ${{ steps.gh_app_token.outputs.token }} | |||
persist-credentials: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is just so we can easily find which repos are using the insecure method, correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes it lets us find them and also makes it clear to everyone looking at the workflow that it's the case, basically raising awareness
Change-type: patch
Contributor checklist
Change-type
present on at least one commitSigned-off-by
is presentReviewer Guidelines