Fix PRESUBMIT errors of catapult |
||
Issue descriptionWith the new pylint roll, we now have a bunch of new PRESUBMIT errors. Open this bug to fix them.
,
Jul 23
,
Jul 23
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/84c1e861467cefc2877f135c1e5c5fe977271475 commit 84c1e861467cefc2877f135c1e5c5fe977271475 Author: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Date: Mon Jul 23 22:17:20 2018 Roll src/third_party/catapult 9e8a0b2b37d9..1381813c377d (3 commits) https://chromium.googlesource.com/catapult.git/+log/9e8a0b2b37d9..1381813c377d git log 9e8a0b2b37d9..1381813c377d --date=short --no-merges --format='%ad %ae %s' 2018-07-23 nednguyen@google.com Increase default timeout limit of setting tsproxy's traffic settings to 20 seconds 2018-07-23 nednguyen@google.com Add '--story-shard-begin-index' and '--story-shard-end-index' flags 2018-07-23 nednguyen@google.com Fix PRESUBMIT errors in catapult Created with: gclient setdep -r src/third_party/catapult@1381813c377d The AutoRoll server is located here: https://catapult-roll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel BUG=chromium:865250, chromium:866095 , chromium:866260 TBR=sullivan@chromium.org Change-Id: I0f4b0a06fa6b8e5312fbbd6f6385910004841801 Reviewed-on: https://chromium-review.googlesource.com/1147060 Reviewed-by: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#577290} [modify] https://crrev.com/84c1e861467cefc2877f135c1e5c5fe977271475/DEPS
,
Jul 24
The following revision refers to this bug: https://chromium.googlesource.com/catapult/+/c9111f68cac95234cc8ffb83ae965fd2c6dd8153 commit c9111f68cac95234cc8ffb83ae965fd2c6dd8153 Author: Simon <simonhatch@chromium.org> Date: Tue Jul 24 18:37:39 2018 Disable a bunch of pylint warnings. Bug: chromium:866260 Change-Id: I4191c1ace0a97f255ec7b5aff14fdcb1ad049f25 Reviewed-on: https://chromium-review.googlesource.com/1148700 Reviewed-by: Annie Sullivan <sullivan@chromium.org> Commit-Queue: Simon Hatch <simonhatch@chromium.org> [modify] https://crrev.com/c9111f68cac95234cc8ffb83ae965fd2c6dd8153/common/py_utils/py_utils/refactor/annotated_symbol/reference.py [modify] https://crrev.com/c9111f68cac95234cc8ffb83ae965fd2c6dd8153/common/py_utils/py_utils/refactor/annotated_symbol/import_statement.py [modify] https://crrev.com/c9111f68cac95234cc8ffb83ae965fd2c6dd8153/common/py_utils/py_utils/retry_util_unittest.py [modify] https://crrev.com/c9111f68cac95234cc8ffb83ae965fd2c6dd8153/common/py_utils/py_utils/lock.py [modify] https://crrev.com/c9111f68cac95234cc8ffb83ae965fd2c6dd8153/common/py_utils/py_utils/__init__.py
,
Jul 24
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b893d71f7199a668434c20461bb7cefe309f6a3f commit b893d71f7199a668434c20461bb7cefe309f6a3f Author: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Date: Tue Jul 24 21:30:11 2018 Roll src/third_party/catapult d269b0e87d1e..e0ae2989b096 (3 commits) https://chromium.googlesource.com/catapult.git/+log/d269b0e87d1e..e0ae2989b096 git log d269b0e87d1e..e0ae2989b096 --date=short --no-merges --format='%ad %ae %s' 2018-07-24 benjhayden@chromium.org Add a new ReportTemplate system. 2018-07-24 simonhatch@chromium.org Disable a bunch of pylint warnings. 2018-07-24 nastasoiuf@google.com Initial implementation Created with: gclient setdep -r src/third_party/catapult@e0ae2989b096 The AutoRoll server is located here: https://catapult-roll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel BUG= chromium:866260 TBR=sullivan@chromium.org Change-Id: I7f71d7246cfd6312d73d4ef4dac17b1c3c6af864 Reviewed-on: https://chromium-review.googlesource.com/1147917 Reviewed-by: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#577691} [modify] https://crrev.com/b893d71f7199a668434c20461bb7cefe309f6a3f/DEPS |
||
►
Sign in to add a comment |
||
Comment 1 by bugdroid1@chromium.org
, Jul 23