build(deps): Bump Riverpod to 2.6.1 #506
Conversation
| exec: | | ||
| dart fix --apply lib | ||
| dart run custom_lint --fix | ||
| description: Run dart and custom_lint fixes. |
There was a problem hiding this comment.
Also added custom_lint --fix to run.
| ref | ||
| ..listen(appLifecycleProvider, (prev, next) { | ||
| logger.finest('AppLifecycleState: prev: $prev, next: $next'); | ||
| }) | ||
| ..listen(localesProvider, (prev, next) { | ||
| logger.finest('Locales: prev: $prev, next: $next'); | ||
| }) |
There was a problem hiding this comment.
listentoref.listenforappLifecycleProviderandlocalesProvider` to make sure they are being monitored.
|
|
||
| part 'app_lifecycle_provider.g.dart'; | ||
|
|
||
| @riverpod |
There was a problem hiding this comment.
I removed keepAlive:true since it only needs to survive where I want to monitor.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #506 +/- ##
=======================================
Coverage 23.75% 23.75%
=======================================
Files 13 13
Lines 160 160
=======================================
Hits 38 38
Misses 122 122
Continue to review full report in Codecov by Sentry.
|
| @@ -1,4 +1,4 @@ | |||
| export 'app_lifecycle_state_provider.dart'; | |||
| export 'app_lifecycle_provider.dart'; | |||
There was a problem hiding this comment.
Changed the name of Notifier to AppLifecycleState because it conflicts.
riscait
changed the title
naipaka
left a comment
naipaka
left a comment
There was a problem hiding this comment.
🙌 What's Done
✍️ What's Not Done
🤼 Desired Review Method
Note
It is possible that a reviewer's will may cause a method to be implemented that is not selected.
📝 Additional Notes
Pre-launch Checklist