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

[timomeinen]

From gro...@gmail.com on October 14, 2010 07:33:09

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: http://code.google.com/p/team-piazza/issues/detail?id=36

[timomeinen]

From gro...@gmail.com on April 08, 2011 06:32:54

Just revisiting, apparently this does work, so long as names are configured in main-config.xml.

[timomeinen]

From timomeinen on November 29, 2011 07:12:33

Piazza checks commit message and commit metadata - especially author name.

Status: Fixed
Labels: -Type-Defect Type-Enhancement