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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 306283002: pkg/matcher: cleanup, updates, deprecations, fixes (Closed)

Created:
6 years, 6 months ago by kevmoo
Modified:
6 years, 6 months ago
CC:
reviews_dartlang.org
Visibility:
Public.

Description

pkg/matcher: cleanup, updates, deprecations, fixes R=sigmund@google.com Committed: https://code.google.com/p/dart/source/detail?r=36881

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+1500 lines, -1693 lines) Patch
A pkg/matcher/CHANGELOG.md View 1 chunk +12 lines, -0 lines 0 comments Download
A pkg/matcher/README.md View 1 chunk +8 lines, -0 lines 0 comments Download
M pkg/matcher/lib/matcher.dart View 1 chunk +10 lines, -24 lines 0 comments Download
M pkg/matcher/lib/mirror_matchers.dart View 1 chunk +5 lines, -9 lines 0 comments Download
M pkg/matcher/lib/src/core_matchers.dart View 16 chunks +127 lines, -289 lines 0 comments Download
M pkg/matcher/lib/src/description.dart View 3 chunks +21 lines, -35 lines 0 comments Download
A pkg/matcher/lib/src/error_matchers.dart View 1 chunk +160 lines, -0 lines 0 comments Download
M pkg/matcher/lib/src/expect.dart View 6 chunks +45 lines, -54 lines 0 comments Download
M pkg/matcher/lib/src/future_matchers.dart View 1 chunk +24 lines, -22 lines 0 comments Download
M pkg/matcher/lib/src/interfaces.dart View 1 chunk +45 lines, -62 lines 1 comment Download
M pkg/matcher/lib/src/iterable_matchers.dart View 9 chunks +38 lines, -46 lines 0 comments Download
M pkg/matcher/lib/src/map_matchers.dart View 2 chunks +7 lines, -8 lines 0 comments Download
M pkg/matcher/lib/src/numeric_matchers.dart View 6 chunks +49 lines, -72 lines 0 comments Download
M pkg/matcher/lib/src/operator_matchers.dart View 5 chunks +68 lines, -102 lines 0 comments Download
M pkg/matcher/lib/src/pretty_print.dart View 5 chunks +52 lines, -16 lines 0 comments Download
M pkg/matcher/lib/src/string_matchers.dart View 8 chunks +59 lines, -72 lines 0 comments Download
D pkg/matcher/lib/src/utils.dart View 1 chunk +0 lines, -69 lines 0 comments Download
M pkg/matcher/pubspec.yaml View 1 chunk +1 line, -1 line 0 comments Download
A pkg/matcher/test/core_matchers_test.dart View 1 chunk +211 lines, -0 lines 0 comments Download
A pkg/matcher/test/deprecated_matchers_test.dart View 1 chunk +33 lines, -0 lines 0 comments Download
A pkg/matcher/test/future_matchers_test.dart View 1 chunk +76 lines, -0 lines 0 comments Download
A pkg/matcher/test/iterable_matchers_test.dart View 1 chunk +166 lines, -0 lines 0 comments Download
D pkg/matcher/test/matchers_test.dart View 1 chunk +0 lines, -803 lines 0 comments Download
A pkg/matcher/test/numeric_matchers_test.dart View 1 chunk +152 lines, -0 lines 0 comments Download
A pkg/matcher/test/operator_matchers_test.dart View 1 chunk +28 lines, -0 lines 0 comments Download
A pkg/matcher/test/string_matchers_test.dart View 1 chunk +94 lines, -0 lines 0 comments Download
M pkg/matcher/test/test_common.dart View 3 chunks +8 lines, -8 lines 0 comments Download
M pkg/pkg.status View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
kevmoo
A whole mountain of fun here. There were also matcher tests that were still in ...
6 years, 6 months ago (2014-06-01 21:03:10 UTC) #1
Siggi Cherem (dart-lang)
lgtm! https://codereview.chromium.org/306283002/diff/1/pkg/matcher/lib/src/interfaces.dart File pkg/matcher/lib/src/interfaces.dart (right): https://codereview.chromium.org/306283002/diff/1/pkg/matcher/lib/src/interfaces.dart#newcode5 pkg/matcher/lib/src/interfaces.dart:5: library matcher.interfaces; not for this CL, but what ...
6 years, 6 months ago (2014-06-02 16:47:09 UTC) #2
kevmoo
6 years, 6 months ago (2014-06-02 16:54:55 UTC) #3
Message was sent while issue was closed. 
Committed patchset #1 manually as r36881 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698