Skip to content

Avoid doing git operations after checkout#174

Merged
knuton merged 1 commit into
dividat:mainfrom
yfyf:adapt-to-no-credentials
May 22, 2026
Merged

Avoid doing git operations after checkout#174
knuton merged 1 commit into
dividat:mainfrom
yfyf:adapt-to-no-credentials

Conversation

@yfyf
Copy link
Copy Markdown
Contributor

@yfyf yfyf commented May 22, 2026

Noticed that this breaks builds in private repos, since without credentials it is not possible to fetch the tags.

This is future-proofing the usage of persist-credentials: false. For a public repo, this changes nothing. But if driver were to become a private repo or something in Github's logic were to change to block public reads, this would start failing.

Noticed that this breaks builds in private repos, since without
credentials it is not possible to fetch the tags.

This is future-proofing the usage of `persist-credentials: false`. For a
public repo, this changes nothing. But if `driver` were to become a
private repo or something in Github's logic were to change to block
public reads, this would start failing.
@yfyf yfyf added the reviewable Ready for initial or iterative review. label May 22, 2026
@yfyf
Copy link
Copy Markdown
Contributor Author

yfyf commented May 22, 2026

Release tag run on my fork: https://github.com/yfyf/driver/actions/runs/26278423003

Copy link
Copy Markdown
Member

@knuton knuton left a comment

Choose a reason for hiding this comment

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

Good, thanks

@knuton knuton merged commit db2b9c0 into dividat:main May 22, 2026
7 checks passed
@knuton knuton removed the reviewable Ready for initial or iterative review. label May 22, 2026
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.

2 participants