Allow latest lints versions, fix new reported issues

Author: greenrobot-team

benchmark/pubspec.yaml

@@ -11,7 +11,6 @@ dependencies:
   ffi: ^2.0.1
 
 dev_dependencies:
-  # lints 2.1.0+ requires Dart 3, but CI still tests with Dart 2, but also need 3.0.0+ to match pana
-  lints: '>=2.0.1 <4.0.0'
+  lints: '>=5.0.0 <7.0.0' # 6.0.0 requires Dart 3.8, but still testing with 3.7
   objectbox_generator: any
   build_runner: any

generator/pubspec.yaml

@@ -24,7 +24,7 @@ dependencies:
 
 dev_dependencies:
   test: ^1.25.8
-  lints: ^3.0.0
+  lints: '>=5.0.0 <7.0.0' # 6.0.0 requires Dart 3.8, but still testing with 3.7
   pub_semver: ^2.2.0
   package_config: ^2.2.0
   build_test: ^3.0.0

generator/test/generator_test_env.dart

@@ -104,7 +104,7 @@ class GeneratorTestEnv {
 
   /// Check that the model is specified and written to JSON correctly.
   /// Note: there are tests asserting the generated model code in integration-tests/common.dart
-  _commonModelTests(ModelInfo generatorModel, ModelInfo jsonModel) {
+  void _commonModelTests(ModelInfo generatorModel, ModelInfo jsonModel) {
     // collect UIDs on all entities and properties
     final allUIDs = generatorModel.entities
         .map(
@@ -186,11 +186,11 @@ class GeneratorTestEnv {
     expect(jsonModel.version, generatorModel.version);
   }
 
-  _expectIdUid(IdUid actual, IdUid expected) {
+  void _expectIdUid(IdUid actual, IdUid expected) {
     expect(actual.toString(), expected.toString());
   }
 
-  _testLastId(IdUid last, Iterable<IdUid> all, Iterable<int> retired) {
+  void _testLastId(IdUid last, Iterable<IdUid> all, Iterable<int> retired) {
     if (last.isEmpty) return;
 
     // If among used IDs, UID should match; ID and UID not re-used elsewhere

objectbox/lib/src/native/bindings/nativemem.dart

@@ -1,7 +1,7 @@
 import 'dart:ffi';
 import 'dart:io';
 
-/// Provides native memory manipulation, operating on FFI Pointer<Void>.
+/// Provides native memory manipulation, operating on FFI `Pointer<Void>`.
 
 /// memset(ptr, value, num) sets the first num bytes of the block of memory
 /// pointed by ptr to the specified value (interpreted as an uint8).

objectbox/lib/src/relations/to_many.dart

@@ -53,7 +53,7 @@ class ToMany<EntityT> extends Object with ListMixin<EntityT> {
   /// Store which contains a pointer.
   ///
   /// Using dynamic for the owning entity type as adding it would require
-  /// a breaking API change to ToMany (-> ToMany<EntityT, OwningEntityT>).
+  /// a breaking API change to ToMany (-> `ToMany<EntityT, OwningEntityT>`).
   _ToManyStoreConfiguration<EntityT, dynamic>? _storeConfiguration;
 
   List<EntityT>? __items;

objectbox/pubspec.yaml

@@ -20,8 +20,7 @@ dependencies:
 
 dev_dependencies:
   ffigen: ^7.2.11 # v8 requires Dart 3, not requiring it, yet.
-  # lints 2.1.0+ requires Dart 3, but CI still tests with Dart 2, but also need 3.0.0+ to match pana
-  lints: '>=2.0.1 <4.0.0'
+  lints: '>=5.0.0 <7.0.0' # 6.0.0 requires Dart 3.8, but still testing with 3.7
   test: ^1.16.5 # To run coverage on this package.
 
 ffigen:

objectbox_test/pubspec.yaml

@@ -15,8 +15,7 @@ dev_dependencies:
   collection: any # via objectbox
   build_runner: ^2.0.0
   flat_buffers: any # via objectbox
-  # lints 2.1.0+ requires Dart 3, but CI still tests with Dart 2, but also need 3.0.0+ to match pana
-  lints: '>=2.0.1 <4.0.0'
+  lints: '>=5.0.0 <7.0.0' # 6.0.0 requires Dart 3.8, but still testing with 3.7
   objectbox_generator: any
   test: ^1.16.5