Skip to content

feature/add-write-read-tree-entry - #27

Merged
aman-a-shah merged 2 commits into
mainfrom
m1-tree-object-serialization
Sep 21, 2026
Merged

aman-a-shah merged 2 commits into
mainfrom
m1-tree-object-serialization

Conversation

@mahis1067

Copy link
Copy Markdown
Collaborator

Summary

  • Created ObjectStore.write_tree and read_tree
  • Created helper methods to validate tree format
  • Test functions

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

  • Pytest parameters are manually written to check multiple cases of incorrectly formatted tree enteries and hashes

@mahis1067 mahis1067 linked an issue Sep 16, 2026 that may be closed by this pull request
4 tasks
@aman-a-shah
aman-a-shah merged commit e11fda3 into main Sep 21, 2026
1 check passed
@aman-a-shah
aman-a-shah deleted the m1-tree-object-serialization branch September 21, 2026 18:57
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.

Week 3 - M1: Tree object serialization

2 participants