anubhav94 / team-piazza

Automatically exported from code.google.com/p/team-piazza
GNU General Public License v3.0
0 stars 0 forks source link

Read user from commit metadata, and show it in the output #36

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Currently the user is only read from the commit messages, as per the 
documentation at 
http://code.google.com/p/team-piazza/wiki/ConfiguringTeamPortraits. 

Requiring the user to add their username is hard to enforce, and (IMHO) seems 
like something that a tool like Team Piazza should be dictating. The user that 
checked in the change is already part of the commit metadata, Team Piazza 
should pick that up -- and optionally also parse usernames from the commit 
message to retain existing functionality.

This way multiple additional workflows are allowed:
 * No users entering commit messages 
 * Pair programming, with one user committing and making a comment that it was also checked by a second user

In addition to the currently-expected methods by team piazza:
 * Where the user doing the commit types their name in the commit message 
 * Pair programming, where both users put their name in the commit message (the userid as part of the commit will simply be a duplicate)

The userid from the commit message should be displayed before the commit 
message itself, eg:

(greg) Did a change to some code

Original issue reported on code.google.com by gro...@gmail.com on 14 Oct 2010 at 5:33

GoogleCodeExporter commented 9 years ago
Just revisiting, apparently this does work, so long as names are configured in 
main-config.xml.

Original comment by gro...@gmail.com on 8 Apr 2011 at 1:32

GoogleCodeExporter commented 9 years ago
Piazza checks commit message and commit metadata - especially author name.

Original comment by timomeinen on 29 Nov 2011 at 3:12