◐ Shell
reader mode source ↗
Skip to content

Preliminary support for index format v3#2081

Merged
Byron merged 3 commits into
gitpython-developers:mainfrom
blahgeek:support-index-v3
Nov 9, 2025
Merged

Preliminary support for index format v3#2081
Byron merged 3 commits into
gitpython-developers:mainfrom
blahgeek:support-index-v3

Conversation

@blahgeek

@blahgeek blahgeek commented Nov 2, 2025

Copy link
Copy Markdown
Contributor

@Byron Byron left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hide comment

Thanks a lot for tackling this!

I think it would be useful to add a small test-index directly, one that is known to be V3. The Gitoxide repository definitely has them: https://github.com/GitoxideLabs/gitoxide/blob/ecd354119c3a8b150a06df7205ddf022a825d6cd/gix-index/tests/fixtures/loose_index/extended-flags.git-index

Thanks again.

@blahgeek

blahgeek commented Nov 8, 2025

Copy link
Copy Markdown
Contributor Author

I think it would be useful to add a small test-index directly, one that is known to be V3. The Gitoxide repository definitely has them: https://github.com/GitoxideLabs/gitoxide/blob/ecd354119c3a8b150a06df7205ddf022a825d6cd/gix-index/tests/fixtures/loose_index/extended-flags.git-index

OK. I've copied the file here and added a test case

@blahgeek blahgeek requested a review from Byron November 8, 2025 06:08

@Byron Byron left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hide comment

Thanks a lot for adding the tests, it looks great now!

Hide details View details @Byron Byron merged commit 564a043 into gitpython-developers:main Nov 9, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants