Skip to content

week 1 module 3 add CommitManager skeleton and CLI commands - #7

Merged
aman-a-shah merged 15 commits into
mainfrom
week1/m3-commit-manager-skeleton
Sep 2, 2026
Merged

week 1 module 3 add CommitManager skeleton and CLI commands#7
aman-a-shah merged 15 commits into
mainfrom
week1/m3-commit-manager-skeleton

Conversation

@DanisLol

Copy link
Copy Markdown
Member

Summary

  • Added 'CommitManager' skeleton
  • Wired 'minigit commit', 'minigit branch', and 'minigit checkout'
  • Added tests/test_commits.py with fake store and tree for testing

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

  • CommitManager() doesn't have default store so 'minigit commit' will throw an error. Will work when M1 has ObjectStore

@DanisLolDanisLol self-assigned this Aug 31, 2026
@DanisLolDanisLol added the module-3 Commits and branching label Aug 31, 2026
@DanisLolDanisLol linked an issue Aug 31, 2026 that may be closed by this pull request
7 tasks
@aman-a-shah
aman-a-shah merged commit 3865ec1 into mainSep 2, 2026
1 check passed
@aman-a-shah
aman-a-shah deleted the week1/m3-commit-manager-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

module-3Commits and branching

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Week 1 - M3: CommitManager skeleton + commit format

5 participants

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