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

Issue 2784363003: Use the *correct* type for the fix to analysis_server/test/mocks.dart. (Closed)

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

Description

Use the *correct* type for the fix to analysis_server/test/mocks.dart. It's a stream, not an iterable, so the return type is a Future. Committed: https://github.com/dart-lang/sdk/commit/d4c1531b5b3af4f22a4cf0a2229af09ecf715613

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M pkg/analysis_server/test/mocks.dart View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 5 (2 generated)
Lasse Reichstein Nielsen
TBR
3 years, 8 months ago (2017-03-31 14:35:58 UTC) #2
Lasse Reichstein Nielsen
Committed patchset #1 (id:1) manually as d4c1531b5b3af4f22a4cf0a2229af09ecf715613 (presubmit successful).
3 years, 8 months ago (2017-03-31 14:38:01 UTC) #4
Brian Wilkerson
3 years, 8 months ago (2017-03-31 14:44:06 UTC) #5
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698