Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integration with GitWrapper #14

Open
cpliakas opened this issue Feb 26, 2014 · 1 comment
Open

Integration with GitWrapper #14

cpliakas opened this issue Feb 26, 2014 · 1 comment

Comments

@cpliakas
Copy link

Great project!

Just an FYI, I saw that you are executing a lot of Git commands in the codebase. I wanted to put https://github.com/cpliakas/git-wrapper on your radar as it might help to improve reliability and ease development of the application.

More specifically, it can help with logging and exception / error handling as I noticed there were some TODOs around the latter. It could also help facilitate testing since you can bypass execution of the commands if you want to test function in isolation of actually running them.

Anyways, wanted to put it on your radar in case it might be helpful to you.

Either way, great initiative!
Chris

@jonpugh
Copy link
Member

jonpugh commented Feb 26, 2014

Hey, thanks!

That does look pretty great, I'll look into it next time I get some hours to work on hubdrop 😉

Pull requests welcome!

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

No branches or pull requests

2 participants