New issue
Advanced search Search tips

Issue 609492 link

Starred by 1 user

Issue metadata

Status: Archived
Owner: ----
Closed: Jan 3
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Feature



Sign in to add a comment

decrease Cronet pylint ignore list

Project Member Reported by pauljensen@chromium.org, May 5 2016

Issue description

I added pylint presubmit check for Cronet here:
https://codereview.chromium.org/1945023004/

This at least seems to verify the python isn't horribly broken (e.g. functions are called with the right number of parameters), which should have avoided the need for https://codereview.chromium.org/1942283002/.
However we would probably benefit from widdling down the "disable=" list in components/cronet/pylintrc.  I fixed some of the easy pylint failures in https://codereview.chromium.org/1945023004/ but the disable= list is still large.
 
Project Member

Comment 1 by sheriffbot@chromium.org, May 5 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been available for more than 365 days, and should be re-evaluated. Please re-triage this issue.
The Hotlist-Recharge-Cold label is applied for tracking purposes, and should not be removed after re-triaging the issue.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Archived (was: Untriaged)

Sign in to add a comment