1136924 - Remove the revision SHA from the push header

Status: RESOLVED FIXED

(Tree Management :: Treeherder, defect, P3)

Description

[Ed Morley [:emorley]]

Once bug 1136918 makes copying a SHA from the list of commits just _work_, we can remove the somewhat out of place looking SHA from the push header row, to streamline the UI further.

Comment 1

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Attachment: Link to PR

Hrm, this PR ended up comparing against my own fork...

Bug 1136918 should be landed first.

Comment 2

[Ed Morley [:emorley]]

Comment on attachment 8626448 [details] [review]
Link to PR

Left a comment on the PR :-)

Comment 3

[Ed Morley [:emorley]]

This is also virtually ready to land, just needs rebasing and one further removal. Do you want me to get it over the finish line for you?

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Attachment: PR 879

Here we go.

Comment 5

[Ed Morley [:emorley]]

Comment on attachment 8649390 [details] [review]
PR 879

Looks great, thank you :-)

Comment 6

[Treeherder GitHub Bugbot]

Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/28fde2b92e3e2d906cc920575051614b9460d2e7
Bug 1136924 - Remove the revision SHA from push headers