Skip to content

bugfix: Fetch the LFS objects of a checkout of a tag or commit - #160

Merged
seeraven merged 1 commit into
seeraven:masterfrom
mering:bugfix/lfs-non-branch
Sep 11, 2026
Merged

seeraven merged 1 commit into
seeraven:masterfrom
mering:bugfix/lfs-non-branch

Conversation

@mering

@mering mering commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

The refs given to a git checkout command were resolved with git show-ref and an LFS fetch of the mirror was performed only if the ref was found as a remote branch. Therefore, a checkout of a tag or a commit did not fetch the LFS objects into the mirror and the git-lfs filters of the checkout had to download them on their own.

The refs are now resolved to commits and are checked against the mirror, so that a checkout of a tag or a commit fetches the LFS objects as well.

The refs given to a git checkout command were resolved with 'git show-ref'
and an LFS fetch of the mirror was performed only if the ref was found as a
remote branch. Therefore, a checkout of a tag or a commit did not fetch the
LFS objects into the mirror and the git-lfs filters of the checkout had to
download them on their own.

The refs are now resolved to commits and are checked against the mirror, so
that a checkout of a tag or a commit fetches the LFS objects as well.

Co-authored-by: Cursor <cursoragent@cursor.com>
@seeraven seeraven self-assigned this Sep 11, 2026

@seeraven seeraven left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thank you for the PR. It looks good to me and if CI says yes I'm happy to merge it. I just commented on the CHANGELOG.md file if you don't want to add yourself for the credits. ;-)

Comment thread CHANGELOG.md Resolved
@mering mering closed this Sep 11, 2026
@mering mering reopened this Sep 11, 2026
@seeraven
seeraven merged commit 15f8508 into seeraven:master Sep 11, 2026
20 checks passed
@mering
mering deleted the bugfix/lfs-non-branch branch September 11, 2026 15:13
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