Opened on 03/11/2016 at 12:35:03 PM

Closed on 03/11/2016 at 03:07:22 PM

#3784 closed defect (fixed)

[hgreview] Review title is too long, description is cut off

Reported by: fhd Assignee: trev
Priority: P3 Milestone:
Module: Unknown Keywords: hgreview
Cc: trev Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):



hgreview extension revision 2e155c15c704

How to reproduce

  1. Commit something with a commit message in the canonical format:

Noissue - This is the short title

This is the body of the commit message with more information.

  1. Upload a review for this change with hg review -c ...

Observed behaviour

  1. The title of the uploaded review contains not only the first line of the commit message, but the entire message. It gets truncated after 100 characters.
  2. The description of the uploaded review contains the entire commit, but truncated after (presumably) 100 characters.

Expected behaviour

  1. The title of the review should just be the first line of the commit message.
  2. The description of the review should be the entire commit message (including the first line), untruncated.

Attachments (0)

Change History (8)

comment:1 Changed on 03/11/2016 at 01:28:06 PM by trev

That's an or Rietveld bug actually. We should be able to work around it by always setting -m parameter if it isn't given.

comment:2 Changed on 03/11/2016 at 01:33:27 PM by fhd

Don't think it's a bug. Seems like has logic for using the title for both title and description if only --title is specified. And since the title has to be truncated to fit...

IIRC, automatically sets the title to the first line of the message if only --message is specified, so that should do.

Last edited on 03/11/2016 at 01:42:20 PM by fhd

comment:3 Changed on 03/11/2016 at 01:51:42 PM by trev

We'll have to use both anyway, see #3779. And - no, I don't like relying on implicit behavior.

comment:4 Changed on 03/11/2016 at 02:01:08 PM by trev

  • Owner set to trev

comment:5 Changed on 03/11/2016 at 02:14:07 PM by trev

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:6 Changed on 03/11/2016 at 02:14:41 PM by trev

  • Keywords hgreview added

comment:7 Changed on 03/11/2016 at 03:06:48 PM by abpbot

A commit referencing this issue has landed:

comment:8 Changed on 03/11/2016 at 03:07:22 PM by trev

  • Resolution set to fixed
  • Status changed from reviewing to closed

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from trev.
Note: See TracTickets for help on using tickets.