centrifugal / centrifuge-dart

Dart (Flutter) client SDK for bidirectional communication with Centrifugo and Centrifuge-based server over WebSocket
https://pub.dartlang.org/packages/centrifuge
MIT License
109 stars 34 forks source link

Migrate to null safety #47

Closed FZambia closed 3 years ago

FZambia commented 3 years ago

This issue will track progress of migration.

FZambia commented 3 years ago

Dependencies migrated to null safe versions in 6965d6d28ae1f7fe8b23a50a538f888e7420e1d5

FZambia commented 3 years ago

Updated dev dependencies in 555d30fa2a0ad595a6c2612dd61a299310a577bc

dart pub outdated --mode=null-safety
Showing dependencies that are currently not opted in to null-safety.
[✗] indicates versions without null safety support.
[✓] indicates versions opting in to null safety.

All your dependencies declare support for null-safety.
FZambia commented 3 years ago

An ongoing pr with library code migration: #48

srdjan86 commented 3 years ago

Hello,

Sorry for rushing you, but do you maybe know if there is any date by which we can expect the library to be upgraded to null safety?

Cheers

FZambia commented 3 years ago

@srdjan86 hello, could you help to test out code in #48 ?

srdjan86 commented 3 years ago

Hey @FZambia,

I have been using code from #48 for 2 weeks not in the app that I am developing. I have found no issues.

FZambia commented 3 years ago

@srdjan86 thanks, really appreciate help with this!

@mogol wanted to take a final look, if he doesn't find a time till this weekend we will merge that pr in the current form.

FZambia commented 3 years ago

v0.6.0 released