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

Issue 2309333002: Update DEPS to latest linter (0.1.26). (Closed)

Created:
4 years, 3 months ago by pquitslund
Modified:
4 years, 3 months ago
Reviewers:
Brian Wilkerson
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Update DEPS to latest linter (0.1.26). * Updated tests to use package `test` (#302). * Fixed false positive on `[]=` in `always_declare_return_types` (#300). * New `invariant_booleans` lint. * New `literal_only_boolean_expressions` lint. * Fixed `camel_case_types` to allow `$` in identifiers (#290). BUG= R=brianwilkerson@google.com Committed: https://github.com/dart-lang/sdk/commit/45a4ae042b8901e82061d2cd3f965201dbd05445

Patch Set 1 #

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

Messages

Total messages: 5 (2 generated)
pquitslund
4 years, 3 months ago (2016-09-05 19:00:55 UTC) #2
Brian Wilkerson
lgtm
4 years, 3 months ago (2016-09-06 13:47:29 UTC) #3
pquitslund
4 years, 3 months ago (2016-09-06 23:39:43 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
45a4ae042b8901e82061d2cd3f965201dbd05445 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698