Comments

• haraldmartin Mon Apr 14 14:54:36 -0700 2008

  I think we need better a bit more expressive commit messages than these “Merge branch ‘master’ of git://github.com/...”. Isn’t that possible when pushing ?

• josh Mon Apr 14 15:59:45 -0700 2008

  The real commit is here – http://github.com/josh/rails/commit/89237fe07e9c6d35687b3a473f09f4608a3625d8
  
  And yes we are still working out a commit workflow to get the core commits name and author of the patch to show up in the commit log. If you have any good insight please message me.

• technoweenie Mon Apr 14 18:04:25 -0700 2008

  I think it’s fine, unless we can somehow get this commit message to link to the relevant commits that were merged in.

• up_the_irons Mon Apr 14 18:17:33 -0700 2008

  Tip: “git merge” takes a ‘-m’ option just like commit, so you can provide a more verbose commit message if needed.

• josh Mon Apr 14 18:20:53 -0700 2008

  Rick, parent links to the previous commit and the merged commit.