-
Notifications
You must be signed in to change notification settings - Fork 9.7k
[nnbd] Various fixes #3175
[nnbd] Various fixes #3175
Conversation
blasten
commented
Oct 21, 2020
- Cleans up some of the code, so it's consistent.
- Fixes some checks. (hopefully all)
| // found in the LICENSE file. | ||
|
|
||
| // @dart = 2.8 | ||
| // @dart = 2.9 |
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.
connectivity hasn't moved to nnbd yet, my migration PR will handle this. Do you want to maybe revert it for now?
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.
@dart = 2.9 is non-null safety. @dart = 2.10 enables null safety.
|
|
||
| ## 0.0.1+1 | ||
|
|
||
| * Update README to reflect endorsement. |
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.
Duplicated 0.0.1+1. Looks like a merge error.
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.
Good catch
|
Some dependency resolution will fail for this patch, but it should be fixed after this patch is merged. This is due to some git dependencies depending on path dependencies. After this change, everything should be fixed. |
cyanglaz
left a comment
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.
LGTM