-
Notifications
You must be signed in to change notification settings - Fork 33
[null-safety] migrate package:process to _almost_ null safety #47
Conversation
path: ^1.5.1 | ||
platform: '>=1.0.1' | ||
|
||
dev_dependencies: | ||
test: ^1.0.0 | ||
|
||
environment: | ||
sdk: '>=2.0.0 <3.0.0' | ||
sdk: '>=2.9.0 <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.
Not 2.10.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.
Ahh right, we switched versions. 😄
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.
actually we're currently at 2.9 and the next version is null safety. I think flutter is on an allowlist to specify a 2.10 constraint which this package isn't on yet
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.
ack
@@ -53,15 +54,14 @@ class ProcessWrapper implements io.Process { | |||
/// | |||
/// The future returned here will complete with the exit code of the process. | |||
Future<int> get done async { | |||
int result; | |||
late int result; |
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 love this feature.
Why was record/replay removed? README still advertises the package as having this feature. |
Migrate to null-safety, except for the path/file/platform dependencies which are not yet migrated.
Removes record/replay functionality.