Skip to content

Only generate a checkout token on private repositories#18

Merged
flowzone-app[bot] merged 6 commits into
masterfrom
kyle/fix-fetch-depth
Jun 5, 2025
Merged

Only generate a checkout token on private repositories#18
flowzone-app[bot] merged 6 commits into
masterfrom
kyle/fix-fetch-depth

Conversation

@klutchell

@klutchell klutchell commented Jun 5, 2025

Copy link
Copy Markdown
Collaborator

Manual testing of product-os#1505

klutchell added 5 commits June 5, 2025 08:55
We don't need submodules to run versionist so we can
save time and skip this.

CHange-type: patch
Signed-off-by: Kyle Harding <kyle@balena.io>
Unfortunately artifact download on self-hosted runners is
very slow compared to git checkouts, so after some use in production
we decided to roll back this change.

Still we can use a dedicated token for the checkout step with fixed
permissions.

Change-type: minor
Signed-off-by: Kyle Harding <kyle@balena.io>
Public repositories should use the automatic github.token.

Change-type: patch
Signed-off-by: Kyle Harding <kyle@balena.io>
This rule is now duplicated by CodeQL actions/untrusted-checkout/high

Change-type: patch
Signed-off-by: Kyle Harding <kyle@balena.io>
Signed-off-by: Kyle Harding <kyle@balena.io>
@klutchell klutchell force-pushed the kyle/fix-fetch-depth branch from 146bf3d to 8e375f9 Compare June 5, 2025 16:23
Signed-off-by: Kyle Harding <kyle@balena.io>
@klutchell

Copy link
Copy Markdown
Collaborator Author

I self-certify!

@flowzone-app flowzone-app Bot merged commit 11c0bc3 into master Jun 5, 2025
68 checks passed
@flowzone-app flowzone-app Bot deleted the kyle/fix-fetch-depth branch June 5, 2025 16:38
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