-
Notifications
You must be signed in to change notification settings - Fork 357
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
Allow the null safety release of stream_transform #1223
Conversation
@nex3 ? |
|
@@ -25,7 +25,7 @@ dependencies: | |||
source_maps: "^0.10.5" | |||
source_span: "^1.6.0" | |||
stack_trace: ">=0.9.0 <2.0.0" | |||
stream_transform: ">=0.0.20 <2.0.0" | |||
stream_transform: ">=0.0.20 <3.0.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We just published the stable release of stream_transform
. It'll be picked up by a fresh pub get
.
@nex3 – if you restart the tests here, we can confirm it's picked up on the Dart dev
builds to ensure nothing is broken.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I rebased the branch on top of master so the tests got restared as well. The new stable stream_transform
was picked and it seems that everything works fine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nex3 ?
9f21b8e
to
1fba2c0
Compare
1fba2c0
to
1c5a3a5
Compare
LGTM, thanks. |
This advances the scanner to the next line even if the !default identifier is not followed by a comma.
No description provided.