Skip to content
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

gix-protocol::fetch() without Delegates #1634

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

Byron
Copy link
Member

@Byron Byron commented Oct 18, 2024

Having learned from gix it should now be possible to have a command-oriented fetch
that is similar to gix_protocol::ls_refs().

Motivated by #1319, as these changes should allow heartwood to use most of gix-protocol
without duplicating a lot of code.

Tasks

  • make current fetch() test-only
  • re-create fetch() as command similar to ls_refs(), based on gix
  • re-implement the test-only fetch with the command-oriented fetch, maybe provide it
    as utility in gix-protocol even? That would allow the Delegates to remain, but
    be specific to this particularly 'simple' way of performing a fetch.
  • let gix use the command-oriented fetch.

@Byron Byron mentioned this pull request Oct 18, 2024
2 tasks
This is done with a test-only function that is siimlar the current `fetch()`
one, performing the whole operation from start to finish, while allowing
to override individual steps just enough to make it work.
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