My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 267: patch set messages can only be one-liners
1 person starred this issue and may be notified of changes. Back to list
Status:  New
Owner:  ----


Sign in to add a comment
 
Reported by paolo.bo...@gmail.com, Jan 27, 2011
Many projects split issues in many small commits.  These can be easily represented as patch sets (one per commit in the series).

However, in this case it is impossible to carry a full-blown description for each commit.  Having a textarea field available for the patch set would be useful.

Jan 27, 2011
Project Member #1 albrecht.andi
Actually the intention of patch sets is to represent different (and evolving) versions of the same changeset. They can't be easily used for the purpose you described. That would break a lot (e.g. how navigation works, the delta between patchsets).

I'm closing this as WontFix (let's take this to the mailing list, if I'm too quick here).
Status: WontFix
Jan 28, 2011
#2 paolo.bo...@gmail.com
Uhm, I knew I shouldn't have mentioned my use case. ;)

I still believe a longer description box can be useful as it ends up in a comment.  Sometimes, a single line is not enough to say what changed between revisions (for example if you want to reply to many comments at once).  Even if you sneak it in the one-line box, the presentation in boldface text is not optimal.  Maybe this can be reconsidered even for rietveld's more common svn-based usage?
Jan 28, 2011
Project Member #3 albrecht.andi
We've got the "-d/--description" command line flag. I think there's currently some misbehavior when updating an issue. IMO the description show up in the web interface somewhere and "-m/--message" should be kept as a one-liner. Is it that what you're thinking of?
Status: New
Jan 28, 2011
#4 paolo.bo...@gmail.com
Maybe the -d is missing in the web interface to add a new patchset then?
Apr 6, 2012
Project Member #5 albrecht.andi
(No comment was entered for this change.)
Labels: Component-Server
Sign in to add a comment

Powered by Google Project Hosting