Skip to content

Module 4: remote client skeleton - #9

Merged
aman-a-shah merged 21 commits into
mainfrom
week1/m4-remote-client-skeleton
Sep 2, 2026
Merged

Module 4: remote client skeleton#9
aman-a-shah merged 21 commits into
mainfrom
week1/m4-remote-client-skeleton

Conversation

@Shuhan6017

Copy link
Copy Markdown
Collaborator

Summary

Add the Module 4 RemoteClient skeleton: push/pull method stubs
Add _parse_address with colon splitting, and alphabet and number confirmation (used for host and ports)
push/pull into the CLI (note: not entirely sure how it works, done with the help with AI)
Add tests against FakeObjectStore/`FakeCommitManager'

Validation

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

Checklist

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

Notes for Reviewers

Might not be applicable, but I wasn't aware of the pre-commit hooks, so there might be some missing commits.
I commented out ObjectStore/CommitManager . I'll have to double check what it is called with devs in module 1 & 3.
thus, the fallback wouldn't work.

@aman-a-shah
aman-a-shah merged commit 6f63fbe into mainSep 2, 2026
1 check passed
@aman-a-shah
aman-a-shah deleted the week1/m4-remote-client-skeleton branch September 2, 2026 19:10
Sign up for freeto 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.

6 participants

@Shuhan6017@aman-a-shah@mahis1067@DanisLol@Saanvi521