Re: [PATCH v3 0/4] Implement git stash as a builtin command

2017-10-23 Thread Joel Teichroeb
Hi Johannes, I fixed all the tests at that time, but another one was added that I could not figure out how to fix. I was planning to break the commit up into smaller parts and only convert one stash command at a time calling them still from the shell script. I haven't had time to do that yet thoug

Re: [PATCH v3 0/4] Implement git stash as a builtin command

2017-10-23 Thread Johannes Schindelin
Hi Joel, On Sun, 28 May 2017, Joel Teichroeb wrote: > I've rewritten git stash as a builtin c command. All tests pass, > and I've added two new tests. Test coverage is around 95% with the > only things missing coverage being error handlers. I am embarrassed to say that I never found the time to

Re: [PATCH v3 0/4] Implement git stash as a builtin command

2017-05-28 Thread Ævar Arnfjörð Bjarmason
On Sun, May 28, 2017 at 6:56 PM, Joel Teichroeb wrote: > I've rewritten git stash as a builtin c command. All tests pass, > and I've added two new tests. Test coverage is around 95% with the > only things missing coverage being error handlers. Worth noting, with your patches the best of 3 run of

[PATCH v3 0/4] Implement git stash as a builtin command

2017-05-28 Thread Joel Teichroeb
I've rewritten git stash as a builtin c command. All tests pass, and I've added two new tests. Test coverage is around 95% with the only things missing coverage being error handlers. Joel Teichroeb (4): stash: add test for stash create with no files stash: add test for stashing in a detached s