update-test-expectations should prepare a standard commit message |
||||
Issue descriptionThe update-test-expectations script should probably create a standard commit message. A standardized message would make it easy to tell at a glance what's changed. At a minimum, I think it should: List all removed tests (maybe up to some limited number?) Provide a link to the dashboard showing the tests runs Mark BUG= for all removed lines
,
Apr 3 2017
OK to mark this as available for now? I would imagine that the script would just print the suggested message, then the user could copy and paste that, and it would look something like this: https://codereview.chromium.org/2778843002
,
May 19 2017
I think I'd like to do this before the next time I run update-test-expectations.
,
May 23 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/3abe1699211d5ff8cc44383a01e395ee13d7820c commit 3abe1699211d5ff8cc44383a01e395ee13d7820c Author: qyearsley <qyearsley@chromium.org> Date: Tue May 23 14:52:58 2017 Print a suggested commit description after removing flaky lines. BUG= 653259 Review-Url: https://codereview.chromium.org/2902523002 Cr-Commit-Position: refs/heads/master@{#473893} [modify] https://crrev.com/3abe1699211d5ff8cc44383a01e395ee13d7820c/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/update_test_expectations.py [modify] https://crrev.com/3abe1699211d5ff8cc44383a01e395ee13d7820c/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/update_test_expectations_unittest.py
,
May 25 2017
|
||||
►
Sign in to add a comment |
||||
Comment 1 by qyears...@chromium.org
, Oct 5 2016