Page MenuHomePhabricator

[GOAL] Make it easier to see intentionality and attribution of visual differences on https://pixel.wmcloud.org
Open, Needs TriagePublic

Description

Background

After talking to the developers on the web team in T317675, one of the leading pain points that emerged was that https://pixel.wmcloud.org lacks a way to see attribution (who/what caused changes?) and intentionality (were these changes intentional?) of visual differences.

Currently, if you want to know the context around the failures that appear on https://pixel.wmcloud.org, you need to manually hunt them down yourself by either going through commit messages, Phabricator tasks, or asking people. It can be a pretty big time waster.

This epic aims to make that easier by changing the report displayed on https://pixel.wmcloud.org to also show a list of commits that caused the change (attribution) and display the commit message associated with each change (intentionality) to alleviate some of this pain.

Wireframe

pixel.png (1×1 px, 261 KB)

Acceptance Criteria

  • When failures are displayed on the report comparing the latest release branches to the HEAD of the master branches, there are also a list of commits associated with those failures.
  • Clicking on a "failed" commit should take you to the report generated for that commit.
  • The commit message including its author should be displayed on each "failed" commit's report.

Event Timeline

nray renamed this task from [Goal] Make it easier to see intentionality and attribution of failures on https://pixel.wmcloud.org to [Goal] Make it easier to see intentionality and attribution of visual differences on https://pixel.wmcloud.org .Oct 8 2022, 12:15 AM
nray renamed this task from [Goal] Make it easier to see intentionality and attribution of visual differences on https://pixel.wmcloud.org to [GOAL] Make it easier to see intentionality and attribution of visual differences on https://pixel.wmcloud.org .Oct 8 2022, 12:17 AM
nray updated the task description. (Show Details)
nray closed this task as a duplicate of T317669: Allow for expected regressions.
nray added a subscriber: Jdlrobson.
nray updated the task description. (Show Details)