Migrate to null-safety (#1029)

* migrate platform interface to null safety

* bump dependencies used by platform interface

* update cirrus script to use beta channel

* change version number used for initial null safety release for platform interface

* updated constraints for plugin_platform_interface dependency

* remove obsolete todo comment in platform interface tests

* fix PendingNotificationRequest class so that title, body and payload are nullable

* migrate plugin to null safety and address deprecation warnings for buttons

* update entitlements for macos example app to allow for network calls

* fix cirrus script to build ios example app on beta channel

* fix version mentioned in changelog entry

* update version reqs of dependencies used by plugin_platform_interface

* update platform interface to go back to null safety prelease of mockito

* update cirrus script to use stable channel

* change macos image used in cirrus script

* restore steps to switch to stable channel when building on macos

* bump plugin's dependency on platform interface

* update platform interface stable nnbd release

* bump plugin for nnbd stable release

Author: MaikuB

.cirrus.yml

@@ -11,21 +11,24 @@ task:
 task:
   name: Build iOS example app
   osx_instance:
-    image: catalina-flutter
+    image: big-sur-xcode-12.4
   pub_cache:
     folder: ~/.pub-cache
+  upgrade_script:
+    - flutter channel stable
+    - flutter upgrade
   build_script:
     - cd flutter_local_notifications/example
     - flutter build ios --no-codesign --debug
 
 task:
   name: Build macOS example app
   osx_instance:
-    image: catalina-flutter
+    image: big-sur-xcode-12.4
   pub_cache:
     folder: ~/.pub-cache
   upgrade_script:
-    - flutter channel dev
+    - flutter channel stable
     - flutter upgrade
   setup_script:
     - flutter config --enable-macos-desktop

analysis_options.yaml

@@ -7,7 +7,7 @@ linter:
     - always_specify_types
     - annotate_overrides
     - avoid_annotating_with_dynamic
-    - avoid_as
+    # - avoid_as
     - avoid_bool_literals_in_conditional_expressions
     - avoid_catches_without_on_clauses
     - avoid_catching_errors
@@ -128,7 +128,7 @@ linter:
     - slash_for_doc_comments
     - sort_child_properties_last
     - sort_constructors_first
-    - sort_pub_dependencies
+    # - sort_pub_dependencies
     - sort_unnamed_constructors_first
     - test_types_in_equals
     - throw_in_finally

flutter_local_notifications/CHANGELOG.md

@@ -1,3 +1,7 @@
+# [5.0.0]
+
+* **Breaking change** migrated to null safety. Some arguments that were formerly null (e.g. some boolean values) are now non-nullable with a default value that should retain the old behaviour
+
 # [4.0.1+2]
 
 * [iOS/macOS] fixed issue where not requesting any permissions (i.e. all the boolean flags were set to false) would still cause a permissions prompt to appear. Thanks to the PR from [Andrey Parvatkin](https://github.com/badver)

flutter_local_notifications/example/analysis_options.yaml

@@ -7,7 +7,7 @@ linter:
     - always_specify_types
     - annotate_overrides
     - avoid_annotating_with_dynamic
-    - avoid_as
+    # - avoid_as
     - avoid_bool_literals_in_conditional_expressions
     - avoid_catches_without_on_clauses
     - avoid_catching_errors
@@ -128,7 +128,7 @@ linter:
     - slash_for_doc_comments
     - sort_child_properties_last
     - sort_constructors_first
-    - sort_pub_dependencies
+    # - sort_pub_dependencies
     - sort_unnamed_constructors_first
     - test_types_in_equals
     - throw_in_finally

flutter_local_notifications/example/integration_test/flutter_local_notifications_test.dart

@@ -1,3 +1,5 @@
+// @dart = 2.9
+
 import 'dart:io';
 
 import 'package:flutter_local_notifications/flutter_local_notifications.dart';