New issue
Advanced search Search tips

Issue 886968 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 20
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug
Build-Toolchain



Sign in to add a comment

[toolchain-utils] Update bisect_driver.py for Android toolchain team

Project Member Reported by cmt...@chromium.org, Sep 19

Issue description

We share our bisection tool code with the Android toolchain team.  They have requested that we add two pylint disable options to bisect_driver.py, to make their pylinter happy (they have a different default format than we do).
 
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 20

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/toolchain-utils/+/6b067ab95b15a5aadf5848cd7f2ae9b70c91948b

commit 6b067ab95b15a5aadf5848cd7f2ae9b70c91948b
Author: Caroline Tice <cmtice@google.com>
Date: Thu Sep 20 22:00:27 2018

Fix bisect_driver.py for Android pylint.

The Android toolchain team has requested that we make this
small pylint change to bisect_driver.py

BUG= 886968 
TEST=None.  This does not affect executable code.

Change-Id: Ia6dc37fc1e204c67b260334b19d1a4ec842924ad
Reviewed-on: https://chromium-review.googlesource.com/1234713
Commit-Ready: Caroline Tice <cmtice@chromium.org>
Tested-by: Caroline Tice <cmtice@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>

[modify] https://crrev.com/6b067ab95b15a5aadf5848cd7f2ae9b70c91948b/binary_search_tool/bisect_driver.py

Status: Fixed (was: Untriaged)

Sign in to add a comment