Riverpod 3.0

.changeset/README.md

@@ -0,0 +1,8 @@
+# Changesets
+
+Hello and welcome! This folder has been automatically generated by `@changesets/cli`, a build tool that works
+with multi-package repos, or single-package repos to help you version and publish your code. You can
+find the full documentation for it [in our repository](https://github.com/changesets/changesets)
+
+We have a quick list of common questions to get you started engaging with this project in
+[our documentation](https://github.com/changesets/changesets/blob/main/docs/common-questions.md)

.changeset/config.json

@@ -0,0 +1,11 @@
+{
+  "$schema": "https://unpkg.com/@changesets/config@3.0.0/schema.json",
+  "changelog": "@changesets/cli/changelog",
+  "commit": false,
+  "fixed": [],
+  "linked": [],
+  "access": "restricted",
+  "baseBranch": "main",
+  "updateInternalDependencies": "patch",
+  "ignore": []
+}

.gitattributes

@@ -0,0 +1 @@
+**/*.*.dart linguist-generated=true

.github/workflows/build.yml

@@ -5,13 +5,6 @@ on:
     paths-ignore:
       - "**.md"
       - "**.mdx"
-  push:
-    branches:
-      - master
-      - dev
-    paths-ignore:
-      - "**.md"
-      - "**.mdx"
 
   schedule:
     # runs the CI everyday at 10AM
@@ -28,8 +21,7 @@ jobs:
     strategy:
       matrix:
         channel:
-          # Waiting for WidgetStateProperty to be stable
-          # - stable
+          - stable
           - master
         package_path:
           - examples/counter
@@ -45,8 +37,6 @@ jobs:
           - packages/hooks_riverpod
           - packages/hooks_riverpod/example
           - packages/riverpod_annotation
-          # TODO(rrousselGit) update riverpod_cli test setup to be supported by the CI
-          # - packages/riverpod_cli
           - packages/riverpod_generator
           - packages/riverpod_generator/integration/build_yaml
           # TODO(rrousselGit) update riverpod_graph test setup to be supported by the CI

.github/workflows/check_generation.yml

@@ -2,6 +2,8 @@ name: Check code-generation
 
 on:
   pull_request:
+    paths-ignore:
+      - "**.md"
   push:
     branches:
       - master

.github/workflows/riverpod_lint.yml

@@ -4,12 +4,6 @@ on:
   pull_request:
     paths-ignore:
       - "**.md"
-  push:
-    branches:
-      - master
-      - dev
-    paths-ignore:
-      - "**.md"
   schedule:
     # runs the CI everyday at 10AM
     - cron: "0 10 * * *"
@@ -26,6 +20,7 @@ jobs:
       matrix:
         channel:
           - master
+          - stable
         package_path:
           - packages/riverpod_analyzer_utils
           - packages/riverpod_analyzer_utils_tests

CONTRIBUTING.md

@@ -35,7 +35,7 @@ You do not need to spend a lot of time making that issue. The important is:
 Making an issue is important to discuss what needs to be done.  
 You wouldn't want to spend tens of hours working on something, only to
 have your PR rejected because we disagree on the direction taken.
-An discussion may slow down the work initially, but it can save you from a lot
+A discussion may slow down the work initially, but it can save you from a lot
 of frustration.
 
 ## Documentation work

README.md

@@ -46,7 +46,7 @@ Long story short:
 
   ```dart
   @riverpod
-  Future<String> boredSuggestion(BoredSuggestionRef ref) async {
+  Future<String> boredSuggestion(Ref ref) async {
     final response = await http.get(
       Uri.https('boredapi.com', '/api/activity'),
     );

analysis_options.yaml

@@ -14,6 +14,9 @@ analyzer:
 
 linter:
   rules:
+    # Not an issue if using dartfmt
+    no_adjacent_strings_in_list: false
+
     # False positive for custom enum-like classes (such as Flutter's "Colors")
     avoid_classes_with_only_static_members: false
 
@@ -22,9 +25,6 @@ linter:
 
     # Low value and lacks a quick fix
     combinators_ordering: false
-
-    # Low value and high cost to change on all files
-    eol_at_end_of_file: false
 
     # Conflicts with unused variables
     no_leading_underscores_for_local_identifiers: false
@@ -84,6 +84,3 @@ linter:
 
     # conflicts with `prefer_relative_imports`
     always_use_package_imports: false
-
-    # Disabled for now until we have NNBD as it otherwise conflicts with `missing_return`
-    no_default_cases: false

benchmarks/pubspec.yaml

@@ -4,7 +4,7 @@ publish_to: 'none' # Remove this line if you wish to publish to pub.dev
 version: 1.0.0+1
 
 environment:
-  sdk: ">=2.12.0-0 <4.0.0"
+  sdk: ">=3.0.0<4.0.0"
 
 dependencies:
   flutter:

examples/counter/lib/main.dart

@@ -1,3 +1,5 @@
+// ignore_for_file: unreachable_from_main, https://github.com/dart-lang/linter/issues/4652
+
 import 'package:flutter/material.dart';
 import 'package:flutter_riverpod/flutter_riverpod.dart';
 import 'package:riverpod_annotation/riverpod_annotation.dart';

examples/counter/pubspec.yaml

@@ -24,7 +24,7 @@ dependencies:
 
 dev_dependencies:
   build_runner: ^2.3.3
-  custom_lint: ^0.6.5
+  custom_lint: ^0.7.0
   flutter_test:
     sdk: flutter
   freezed: ^2.3.2

examples/counter/pubspec_overrides.yaml

@@ -1,4 +1,4 @@
-# melos_managed_dependency_overrides: flutter_riverpod,hooks_riverpod,riverpod,riverpod_lint,riverpod_analyzer_utils
+# melos_managed_dependency_overrides: flutter_riverpod,hooks_riverpod,riverpod,riverpod_analyzer_utils,riverpod_lint
 dependency_overrides:
   flutter_riverpod:
     path: ../../packages/flutter_riverpod
@@ -8,9 +8,9 @@ dependency_overrides:
     path: ../../packages/riverpod
   riverpod_analyzer_utils:
     path: ../../packages/riverpod_analyzer_utils
-  riverpod_lint:
-    path: ../../packages/riverpod_lint
   riverpod_annotation:
     path: ../../packages/riverpod_annotation
   riverpod_generator:
     path: ../../packages/riverpod_generator
+  riverpod_lint:
+    path: ../../packages/riverpod_lint

examples/marvel/lib/src/widgets/search_bar.dart

@@ -6,18 +6,17 @@ import 'package:hooks_riverpod/hooks_riverpod.dart';
 
 import '../screens/home.dart';
 
+const _height = 300.0;
+
 class _SearchTheme {
   const _SearchTheme({
     required this.width,
-    // ignore: unused_element, blocked by https://github.com/dart-lang/linter/issues/3232
-    this.height = 300,
     required this.searchDecoration,
     required this.iconPadding,
     required this.searchMargin,
   });
 
   final double width;
-  final double height;
   final BoxDecoration searchDecoration;
   final EdgeInsets iconPadding;
   final EdgeInsets searchMargin;
@@ -187,7 +186,7 @@ class _SearchHintContainer extends StatelessWidget {
   Widget build(BuildContext context) {
     return Container(
       constraints: BoxConstraints(
-        maxHeight: theme.height,
+        maxHeight: _height,
       ),
       margin: theme.searchMargin,
       child: Material(
@@ -196,7 +195,7 @@ class _SearchHintContainer extends StatelessWidget {
         clipBehavior: Clip.hardEdge,
         child: OverflowBox(
           alignment: Alignment.topLeft,
-          maxHeight: theme.height,
+          maxHeight: _height,
           maxWidth: 300,
           child: child,
         ),

examples/marvel/pubspec.yaml

@@ -22,7 +22,7 @@ dependencies:
 
 dev_dependencies:
   build_runner: ^2.0.0
-  custom_lint: ^0.6.5
+  custom_lint: ^0.7.0
   flutter_test:
     sdk: flutter
   freezed: ^2.1.0

examples/marvel/pubspec_overrides.yaml

@@ -1,4 +1,4 @@
-# melos_managed_dependency_overrides: flutter_riverpod,hooks_riverpod,riverpod,riverpod_lint,riverpod_analyzer_utils
+# melos_managed_dependency_overrides: flutter_riverpod,hooks_riverpod,riverpod,riverpod_analyzer_utils,riverpod_lint
 dependency_overrides:
   flutter_riverpod:
     path: ../../packages/flutter_riverpod
@@ -8,9 +8,9 @@ dependency_overrides:
     path: ../../packages/riverpod
   riverpod_analyzer_utils:
     path: ../../packages/riverpod_analyzer_utils
-  riverpod_lint:
-    path: ../../packages/riverpod_lint
   riverpod_annotation:
     path: ../../packages/riverpod_annotation
   riverpod_generator:
     path: ../../packages/riverpod_generator
+  riverpod_lint:
+    path: ../../packages/riverpod_lint