Skip to content

week6-initial-commit - #53

Open
mahis1067 wants to merge 1 commit into
mainfrom
38-week-6---m1-object-store-fixes-for-pushpull-round-trips
Open

mahis1067 wants to merge 1 commit into
mainfrom
38-week-6---m1-object-store-fixes-for-pushpull-round-trips

Conversation

@mahis1067

Copy link
Copy Markdown
Collaborator

Summary

  • complete object-graph transfers.
  • pull now discovers both merge parents, nested trees, and all referenced blobs.
  • added validation for object hashes and expected object types while reusing valid loose or packed objects.
  • added remote integration coverage for packed-only stores, missing descendants, binary/Unicode/empty data, and merge commits.

Validation

  • scripts/quality-check.sh
  • Manual testing, if applicable:

Checklist

  • The change is focused and easy to review.
  • Tests were added or updated for behavior changes.
  • Documentation was updated for user-facing or setup changes.
  • No secrets, generated artifacts, or local-only files are included.

Notes for Reviewers

tests/test_index.py::test_mode_only_change_is_modified_with_no_line_edits
tests/test_index.py::test_cli_mode_only_change

these tests did not pass

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