Skip to content

Add explicit accessors to gix_diff::index::ChangeRef` #2071

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

Merged
merged 4 commits into from
Jul 3, 2025

Conversation

cruessler
Copy link
Contributor

  • feat: add explicit accessors for common fields
  • Adapt to changes in gix-diff

There don’t seem to be any explicit tests specifically for .fields() yet, so I extended an existing test that already used .fields().0. Let me know if there’s a better way to test this!

This PR is based on an idea in this comment: gitui-org/gitui#2673 (comment)

Copy link
Member

@Byron Byron left a comment

Choose a reason for hiding this comment

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

I think that's great, thanks so much :)!

@Byron Byron merged commit 5335c84 into GitoxideLabs:main Jul 3, 2025
23 checks passed
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