Skip to content

Conversation

@lavanyagarg112
Copy link
Contributor

@lavanyagarg112 lavanyagarg112 commented Oct 29, 2025

Exercise Review

Exercise Discussion

#87

Checklist

  • If you require a new remote repository on the Git-Mastery organization, have you created a request for it?
  • Have you written unit tests using repo-smith to validate the exercise grading scheme?
  • Have you tested the download script using test-download.sh?
  • Have you verified that this exercise does not already exist or is not currently in review?
  • Did you introduce a new grading mechanism that should belong to git-autograder?
  • Did you introduce a new dependency that should belong to app?

Copy link
Member

@woojiahao woojiahao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lavanyagarg112 You might want to reference #99 too to compare and see if there's anything that could be the same since the setup is equivalent

Copy link
Member

@woojiahao woojiahao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lavanyagarg112 bumping on this for your changes. Otherwise, most LGTM

@lavanyagarg112
Copy link
Contributor Author

@lavanyagarg112 bumping on this for your changes. Otherwise, most LGTM

So sorry for the delay, will resolve it by end of the day!

Copy link
Member

@woojiahao woojiahao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Last nit!

Copy link
Member

@woojiahao woojiahao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thank you @lavanyagarg112 for your help! I will merge this in now

@woojiahao woojiahao merged commit 1c84c33 into git-mastery:main Nov 20, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Hands-On Discussion] T3L3/hp-pull-remote (Pull from a remote)

2 participants