fix: Fix pana static analyze

.github/workflows/build.yml

@@ -1,12 +1,10 @@
 name: CI
 
 on:
-  # pull_request_target:
-  #   types: [opened, synchronize]
-  #   branches: 
-  #     - master
-  #     # Temporarily allow check on dev/flutter2-iris branch
-  #     - 'dev/flutter2-iris'
+  pull_request_target:
+    types: [opened, synchronize]
+    branches: 
+      - master
 
   push:
     branches: 

analysis_options.yaml

@@ -2,10 +2,10 @@ include: package:flutter_lints/flutter.yaml
 analyzer:
   exclude:
     # Ignore generated files
-    - 'lib/src/*.g.dart'
-    - 'lib/src/generated/*.dart'
-    - 'integration_test_app/**'
-    - 'test/**'
+    - "**/*.g.dart"
+    - lib/src/generated/*.dart
+    - integration_test_app/**
+    - test/**
 
 linter:
   rules:

build.yaml

@@ -0,0 +1,9 @@
+targets:
+  $default:
+    builders:
+      source_gen|combining_builder:
+        options:
+          ignore_for_file:
+          - non_constant_identifier_names
+          - deprecated_member_use_from_same_package
+          - unused_element

lib/src/rtc_channel.dart

@@ -112,6 +112,12 @@ class RtcChannel with RtcChannelInterface {
     });
   }
 
+  // TODO(littlegnal): Dead code to pass pana analyze, will fix it in 5.1.0
+  // ignore: unused_element
+  void _cancelSubscription() {
+    _subscription?.cancel();
+  }
+
   ///
   /// Sets the event handler for the RtcChannel object.
   /// After setting the channel event handler, you can listen for channel events and receive the statistics of the corresponding RtcChannel object.

lib/src/rtc_engine.dart

@@ -220,6 +220,12 @@ class RtcEngine with RtcEngineInterface {
     });
   }
 
+  // TODO(littlegnal): Dead code to pass pana analyze, will fix it in 5.1.0
+  // ignore: unused_element
+  void _cancelSubscription() {
+    _subscription?.cancel();
+  }
+
   ///
   /// Adds event handlers
   /// The SDK uses the RtcEngineEventHandler class to send callbacks to the app. The app inherits the methods of this class to receive these callbacks. All methods in this interface class have default (empty) implementations. Therefore, the application can only inherit some required events. In the callbacks, avoid time-consuming tasks or calling APIs that can block the thread, such as the sendStreamMessage method. Otherwise, the SDK may not work properly.