Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "AsyncSnapshot.data to throw if error or no data" #36618

Merged
merged 1 commit into from
Jul 21, 2019

Conversation

tvolkert
Copy link
Contributor

Reverts #34626

Change needs more discussion, so I'm stopping it from going to dev until the discussion is resolved.

@fluttergithubbot fluttergithubbot added the framework flutter/packages/flutter repository. See also f: labels. label Jul 20, 2019
@tvolkert tvolkert requested a review from Hixie July 20, 2019 20:52
@codecov
Copy link

codecov bot commented Jul 20, 2019

Codecov Report

Merging #36618 into master will increase coverage by 1.18%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #36618      +/-   ##
==========================================
+ Coverage   54.55%   55.74%   +1.18%     
==========================================
  Files         188      188              
  Lines       17432    17421      -11     
==========================================
+ Hits         9510     9711     +201     
+ Misses       7922     7710     -212
Flag Coverage Δ
#flutter_tool 55.74% <ø> (+1.18%) ⬆️
Impacted Files Coverage Δ
packages/flutter_tools/lib/src/cache.dart 44.21% <0%> (-14.87%) ⬇️
packages/flutter_tools/lib/src/version.dart 95.07% <0%> (-2.47%) ⬇️
...ckages/flutter_tools/lib/src/ios/code_signing.dart 96.96% <0%> (-1.52%) ⬇️
packages/flutter_tools/lib/src/compile.dart 74.75% <0%> (-0.34%) ⬇️
.../flutter_tools/lib/src/android/android_device.dart 33.5% <0%> (-0.26%) ⬇️
packages/flutter_tools/lib/src/vmservice.dart 38.97% <0%> (-0.16%) ⬇️
.../flutter_tools/lib/src/fuchsia/fuchsia_device.dart 55.76% <0%> (+0.37%) ⬆️
packages/flutter_tools/lib/src/base/utils.dart 94.5% <0%> (+1%) ⬆️
packages/flutter_tools/lib/src/asset.dart 86.55% <0%> (+4.62%) ⬆️
packages/flutter_tools/lib/src/run_hot.dart 48.68% <0%> (+14.91%) ⬆️
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f5f93d0...24ef0ab. Read the comment docs.

Copy link
Contributor

@collinjackson collinjackson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@tvolkert tvolkert merged commit fbd4bb9 into master Jul 21, 2019
@tvolkert tvolkert deleted the revert-34626-async branch July 21, 2019 05:11
johnsonmh pushed a commit to johnsonmh/flutter that referenced this pull request Jul 30, 2019
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
framework flutter/packages/flutter repository. See also f: labels.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants