riverpod

riverpod copied to clipboard

new lint: Warn if a circular dependency is spotted

2

Bad: ```dart @riverpod int a(ARef ref) => ref.watch(aProvider); ``` Good: ```dart // Not a circular dependency, as the "id" may change @riverpod int a(ARef ref, {required int id}) => ref.watch(aProvider,...

rrousselGit

Update `@Riverpod(dependencies)` to receive the providers instead of the annotated function/class

1

This requires making providers consts, but should be doable.

rrousselGit

Some providers currently use `identical` while some use `==` We should unify this, whatever the decision is. It's probably worth making a benchmark about the cost of using `==`

rrousselGit

Going from `AsyncData(42)` to `AsyncData(42)` shouldn't notify listeners. This originally was optimized but probably doesn't make sense after all. Should StreamProvider do the same?

rrousselGit

Statically detect missing missing scoped provider overrides

1

if we request widgets to do: ```dart // List all the scoped providers they use @Scopes([scopedProvider]) class MyWidget extends ConsumerWidget { build(c, ref) { ref.watch(nonScopedProvider); ref.watch(scopedProvider); } } ``` We...

rrousselGit

Warn if ref.watch is used in a showDialog/showModal without a Consumer

2

Bad: ```dart Button( onPressed: () { showDialog(builder: (_) => ref.watch(...), }, ) ``` Good: ```dart Button( onPressed: () { showDialog(builder: (_) =>Consumer(builder: (_, ref, __) => ref.watch(...) ), }, )...

rrousselGit

Warn if a notifier mock doesn't extend a notifier base class

13

Bad: ```dart class ExampleMock extends Mock implements Example ``` Good: ```dart class ExampleMock extends Notifier with Mock implements Example ```

rrousselGit

cf title Although generally this isn't needed, there isn't really a reason to deny such a feature.

rrousselGit

ref.read vs ref.watch vs ref.listen (racing condition)

3

**Describe what scenario you think is uncovered by the existing examples/articles** https://riverpod.dev/docs/concepts/reading **Describe why existing examples/articles do not cover this case** Section "Reading a Provider" doesn't cover case of using...

eli1stark

Fix typo in select.mdx

1

## Related Issues Changed from **invididual** to **individual** in `docs/advanced/select.mdx` doc ## Checklist Before you create this PR confirm that it meets all requirements listed below by checking the relevant...

vinothvino42