At my job I’m using git-p4 to work locally with some rails code in git and push to perforce. It’s working okay but one issue for me is that we require every commit to perforce to have a code review by someone, and we put the reviewer’s name at the bottom of each commit. For example:
Live changes to histograms
-commonized the histograms views & logic
CR: JamesM
Well when I’m working in the git repo, I don’t know who is going to code review it, so I end up having to add CR: JamesM to several commits. It can be done with rebase -i, but it is several steps per commit. I could use git-notes, but that doesn’t follow the format that we like (it puts Notes: in). Because this is a local repo only, changing the commit history is not a big deal. After some searching I found the way:
git filter-branch --msg-filter 'cat && echo "CR: REVIEWER"' p4/master~1..HEAD
This little beauty will append CR: REVIEWER to all of the commits from the master to the current head (all of the local commits).
6 Comments
Mike,
I found your website via the STL Mobile Dev group and its meeting descriptions page. https://groups.google.com/forum/#!forum/stl-mobile-dev and http://sites.google.com/site/stlmobiledev/meetings/meeting-1 In January 2010 you took part in a Mobile Dev meeting and talked about the Palm Pre. At least, I assume that’s what you did. I wasn’t there, I’m just reading the meeting’s notes. Anyway, I was wondering if you’re still involved with webOS at all. If so, I’d like to invite you to a webOS Social held at The Tap Room on August 20. It’s the first webOS meeting in STL and I’m the ringleader. If you’d like more information please see our Meetup.com page: http://www.meetup.com/HPwebOS/Saint-Louis-MO/
Or just email me.
If you’ve been long-gone from webOS, I’m sorry to have bothered you. But if you’re still involved in webOS I think you’d have a nice time. In less than two months from now there will be a formal HP webOS Developer’s Meeting in STL, in case you’re interested.
Cordially,
Rob
always i used to read smaller content that also clear their
motive, and that is also happening with this post which I am reading at
this place.
Good site you’ve got here.. It’s difficult to find good quality writing like yours these days.
I truly appreciate people like you! Take care!!
I was recommended this web site by way of my cousin.
I’m no longer sure whether or not this submit is written by him as no one else know such distinctive about my difficulty. You’re amazing!
Thank you!
I’m excited to discover this website. I want to to thank you for your time just for this fantastic read!! I definitely really liked every bit of it and i also have you book marked to see new stuff in your website.
It’s nearly impossible to find well-informed people in this particular subject, but you seem like you know what you’re talking about!
Thanks
Post a Comment