Capitalize all user initials in revision list

RESOLVED FIXED

Status

Tree Management
Treeherder
P5
minor
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: jfrench, Assigned: jfrench)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

a year ago
Some user initials in push revision history could be better parsed, for example:

sotaro.ikeda.g@gmail.com registered their user name as just 'sotaro', with no last name. This results in a single lowercase 's' in the revision user push initials, which is smaller and more difficult to read than a typical 'AB' two cap characters for most other users.

So perhaps we can capitalize all leading characters in the userTokens name string when we're building the revisions.
(Assignee)

Updated

a year ago
Assignee: nobody → tojonmz
Status: NEW → ASSIGNED
Created attachment 8876899 [details] [review]
[treeherder] tojon:capitalize-revision-initials > mozilla:master
(Assignee)

Updated

a year ago
Attachment #8876899 - Flags: review?(cdawson)
Made a suggestion / comment in the PR.  Thanks for hitting this!  :)
Attachment #8876899 - Flags: review?(cdawson) → review-
(Assignee)

Comment 3

a year ago
Comment on attachment 8876899 [details] [review]
[treeherder] tojon:capitalize-revision-initials > mozilla:master

Yup, that seems to work fine also in testing :)
Attachment #8876899 - Flags: review- → review?(cdawson)
Comment on attachment 8876899 [details] [review]
[treeherder] tojon:capitalize-revision-initials > mozilla:master

Great!  Thanks for the PR!  :)
Attachment #8876899 - Flags: review?(cdawson) → review+
(Assignee)

Comment 6

a year ago
Marking fixed per above merge.
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.