Skip to content

Conversation

Grokzen
Copy link
Contributor

@Grokzen Grokzen commented Aug 25, 2023

Fixes #24
Fixes #33
Fixes #45
Fixes #56

…from SubGit class into it's own contained class SubGitRepo that enables simpler consumption of data/values

Refactor "subgit status" command to use this new repo handling code
Fixed a logic bug with clone-point
Minor other minor fixes
Removed some unused functions that will be converted later
@Grokzen Grokzen requested a review from holmboe September 1, 2023 12:43
@Grokzen Grokzen marked this pull request as ready for review September 1, 2023 12:44
Copy link
Contributor

@holmboe holmboe left a comment

Choose a reason for hiding this comment

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

Generally this is good but see my comment in #33. I think this should be resorted before merging this.

Change out some prints to proper log messages
Refactor out the Yes/No string convert to a generic helper function
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants