# The following function should check the argument and make sure # it is an integer greater than zero otherwise it throws a really weird error: # ...some example of error...
And then save it to the stash with with a descriptive message (use 'git stash save --patch' if I have other stuff in my working tree):
git stash save "foo.c TODO: check argument is greater than zero"
Then later I use 'git stash pop' to bring my comments back into my working tree, make the fix and commit it. (You can name the TODO stash if you've save other things to the stash after the Todo.)
This is a reverse of the "interrupted workflow" example in the git-stash man page. Instead of interrupting my work for the bug, I'm pushing the bug into the stash so it doesn't interrupt me.
The nice thing is the comment is right in the code where it is relevant. Plus by using the string "TODO" in the message, I can quickly view any stashed todos:
% git stash list | grep -i todo stash@{0}: On develop: foo.c TODO: check argument is greater than zero
Obviously this doesn't replace a real bug tracking system, but it works well for fixes I want to get to "soon" without interrupting my current flow. If it turns out I can't get to it soon, it's easily committed so I don't lose it.
Update 11/29: Someone on HN suggested branches were a better way to do this. I played with it briefly and worked out the following script to use branches (this is a quick hack and missing a lot of error checking, don't even thing about trying to use it): (Update: doesn't quite work as written, you need to use "
git symbolic-ref HEAD
" as described here.)#!/bin/sh msg=$1;shift # Kudos: http://stackoverflow.com/questions/1593051/how-to-programmatically-determine-the-current-checked-out-git-branchbranch=`git name-rev --name-only HEAD`git checkout -b $msg git add -p git commit -m "$msg" git checkout $branch
So this approach would work, and I kinda like the idea, but frankly by the time you polish this script enough to make it robust, you may as well just use a real issue tracker.
No comments:
Post a Comment