Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(141)

Issue 721833003: disable new pylint warnings (Closed)

Created:
6 years, 1 month ago by vapier
Modified:
6 years, 1 month ago
CC:
chromium-reviews, Dirk Pranke, cmp-cc_chromium.org, iannucci+depot_tools_chromium.org
Project:
tools
Visibility:
Public.

Description

disable new pylint warnings People are just ignoring these and bypassing the presubmit entirely, so disable all the warnings until someone feels like shaving these yaks one by one. BUG=chromium:431514 TEST=ran presubmit on build/ and so no more warnings Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=292997

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+27 lines, -1 line) Patch
M pylintrc View 1 chunk +27 lines, -1 line 0 comments Download

Messages

Total messages: 6 (3 generated)
Dirk Pranke
rubber-stamp lgtm to get the presubmit unborked in other trees ...
6 years, 1 month ago (2014-11-19 03:04:04 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/721833003/1
6 years, 1 month ago (2014-11-19 03:04:43 UTC) #5
commit-bot: I haz the power
6 years, 1 month ago (2014-11-19 03:06:45 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=292997

Powered by Google App Engine
This is Rietveld 408576698