Comments on: How to yell at people with GitHub from TextMate http://drnicwilliams.com/2008/05/28/how-to-yell-at-people-with-github-from-textmate/ Ruby makes Rails, Javascript makes Ajax, Dr Nic makes Magic Wed, 17 Mar 2010 04:35:01 +0000 http://wordpress.org/?v=2.9.2 hourly 1 By: seslichat http://drnicwilliams.com/2008/05/28/how-to-yell-at-people-with-github-from-textmate/comment-page-1/#comment-167219 seslichat Sun, 12 Apr 2009 11:52:29 +0000 http://drnicwilliams.com/?p=276#comment-167219 I’ve been told that I could reproduce the commit search with the “git blame” command; I might try to use that as its no doubt better than my algorithm. I’ve been told that I could reproduce the commit search with the “git blame” command; I might try to use that as its no doubt better than my algorithm.

]]> By: Lawrence Pit http://drnicwilliams.com/2008/05/28/how-to-yell-at-people-with-github-from-textmate/comment-page-1/#comment-92609 Lawrence Pit Thu, 29 May 2008 07:17:52 +0000 http://drnicwilliams.com/?p=276#comment-92609 Re: line 103, very strange, for me it ends up at: http://github.com/DocSavage/rails-authorization-plugin/commit/bf4e488e5f50ff8700de557eaf24a6aed56814e3#diff-0 that commit id doesn't exist within that project. I checked it out freshly in a different dir, and then line 103 works fine indeed. Re: line 103, very strange, for me it ends up at:

http://github.com/DocSavage/rails-authorization-plugin/commit/bf4e488e5f50ff8700de557eaf24a6aed56814e3#diff-0

that commit id doesn’t exist within that project. I checked it out freshly in a different dir, and then line 103 works fine indeed.

]]> By: Dr Nic http://drnicwilliams.com/2008/05/28/how-to-yell-at-people-with-github-from-textmate/comment-page-1/#comment-92596 Dr Nic Thu, 29 May 2008 05:56:22 +0000 http://drnicwilliams.com/?p=276#comment-92596 The issue with line 104 is a known issue - I'm just matching the exact string, and not caring about line numbers. Re: line 103 - this goes to the correct commit for me. (same as for line 102's commit) Interesting: line 1 - this goes to the correct commit, but wrong file so I might be pulling file names out of the diff_parent incorrectly (prob to do with files being deleted in that commit). I've been told that I could reproduce the commit search with the "git blame" command; I might try to use that as its no doubt better than my algorithm. The issue with line 104 is a known issue – I’m just matching the exact string, and not caring about line numbers.

Re: line 103 – this goes to the correct commit for me. (same as for line 102’s commit)

Interesting: line 1 – this goes to the correct commit, but wrong file so I might be pulling file names out of the diff_parent incorrectly (prob to do with files being deleted in that commit).

I’ve been told that I could reproduce the commit search with the “git blame” command; I might try to use that as its no doubt better than my algorithm.

]]> By: Lawrence Pit http://drnicwilliams.com/2008/05/28/how-to-yell-at-people-with-github-from-textmate/comment-page-1/#comment-92594 Lawrence Pit Thu, 29 May 2008 05:48:01 +0000 http://drnicwilliams.com/?p=276#comment-92594 @drnic -- sure: git clone git://github.com/DocSavage/rails-authorization-plugin.git open in mate, go to identity.rb, put your cursor on line 102, CTRL-OPTION-CMD-G, option 2 works fine. Put your cursor on line 103 and it will show a 404. On line 104 it will show an unexpected page. @drnic — sure:

git clone git://github.com/DocSavage/rails-authorization-plugin.git

open in mate, go to identity.rb, put your cursor on line 102, CTRL-OPTION-CMD-G, option 2 works fine. Put your cursor on line 103 and it will show a 404. On line 104 it will show an unexpected page.

]]> By: Dr Nic http://drnicwilliams.com/2008/05/28/how-to-yell-at-people-with-github-from-textmate/comment-page-1/#comment-92581 Dr Nic Thu, 29 May 2008 04:22:07 +0000 http://drnicwilliams.com/?p=276#comment-92581 @lawrence - do you have an example error case I can look at? github repo + file + line number? More interactivity with github as a messaging center would be awesome; though I think I'll probably wait til there is more API assistance. Otherwise you're polling all commits to see if a message was posted, I guess. @lawrence – do you have an example error case I can look at? github repo + file + line number?

More interactivity with github as a messaging center would be awesome; though I think I’ll probably wait til there is more API assistance. Otherwise you’re polling all commits to see if a message was posted, I guess.

]]> By: Lawrence Pit http://drnicwilliams.com/2008/05/28/how-to-yell-at-people-with-github-from-textmate/comment-page-1/#comment-92577 Lawrence Pit Thu, 29 May 2008 04:01:11 +0000 http://drnicwilliams.com/?p=276#comment-92577 Feature request: can I get a notification in textmate when a comment is posted to me? ;-) Feature request: can I get a notification in textmate when a comment is posted to me? ;-)

]]> By: Lawrence Pit http://drnicwilliams.com/2008/05/28/how-to-yell-at-people-with-github-from-textmate/comment-page-1/#comment-92576 Lawrence Pit Thu, 29 May 2008 03:59:23 +0000 http://drnicwilliams.com/?p=276#comment-92576 Awesome. Sometimes it doesn't determine the correct URL though (specifically the commit id) when trying to Annotate/Blame/Comment, which may result in landing on an unexpected page or even getting the 404 page. Awesome.

Sometimes it doesn’t determine the correct URL though (specifically the commit id) when trying to Annotate/Blame/Comment, which may result in landing on an unexpected page or even getting the 404 page.

]]> By: Dr Nic http://drnicwilliams.com/2008/05/28/how-to-yell-at-people-with-github-from-textmate/comment-page-1/#comment-92522 Dr Nic Wed, 28 May 2008 22:11:44 +0000 http://drnicwilliams.com/?p=276#comment-92522 <a href="https://github.com/drnic/github-bundle" rel="nofollow">New version</a> now has nice error messages. New version now has nice error messages.

]]> By: Glenn http://drnicwilliams.com/2008/05/28/how-to-yell-at-people-with-github-from-textmate/comment-page-1/#comment-92413 Glenn Wed, 28 May 2008 11:43:25 +0000 http://drnicwilliams.com/?p=276#comment-92413 I'll buy two! I’ll buy two!

]]>