Fix null DataList issue + other null-safety concerns#86
Fix null DataList issue + other null-safety concerns#86pierrea wants to merge 9 commits intoDn-a:null-safetyfrom
DataList issue + other null-safety concerns#86Conversation
pierrea
changed the title
DataList issue + other null-safety concernsDataList issue + other null-safety concerns
| double _width = 0; | ||
|
|
||
| final List<DataList> _list = []; | ||
| final List<DataList?> _list = []; |
|
It worked! Why has this not been merged yet? |
|
cc @Dn-a 🙏 |
pedrox-hs
left a comment
pedrox-hs
left a comment
There was a problem hiding this comment.
Maybe some files could be deleted and added to .gitignore. Look an example.
example/android/app/src/main/java/io/flutter/plugins/GeneratedPluginRegistrant.java
Outdated
Show resolved
Hide resolved
| @@ -438,9 +439,9 @@ class _ItemTagsState extends State<ItemTags> { | |||
| void _singleItem(DataListInherited dataSetIn, DataList? dataSet) { | |||
| dataSetIn.list! | |||
| .where((tg) => tg != null) | |||
There was a problem hiding this comment.
To avoid use of !.
| .where((tg) => tg != null) | |
| ..whereNotNull() |
There was a problem hiding this comment.
Thanks but my IDE complains when using whereNotNull, as if it didn't exist...
There was a problem hiding this comment.
My bad, it is an extension method from the collection package and it is not used in this package, I'm sorry. (and sorry for the very long delay)
pubspec.lock
Outdated
| url: "https://pub.dartlang.org" | ||
| source: hosted | ||
| version: "2.5.0-nullsafety.3" | ||
| version: "2.8.1" |
There was a problem hiding this comment.
It's not recommended commit this file in libraries
|
so how do we use this if this has not been merged yet ?! |
@tony123S, sorry for the delay, I think you already did it, but if anyone needs it too: |
4 participants
This PR fixes #79 and addresses all the other null-safety warnings.
Version proposed:
1.0.1