Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[ci] Fix CI failed caused by using an unsupported NodeJS version #183

Merged
merged 1 commit into from
Oct 29, 2024

[ci] Fix CI failed caused by using an unsupported NodeJS version

9736daf
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Merged

[ci] Fix CI failed caused by using an unsupported NodeJS version #183

[ci] Fix CI failed caused by using an unsupported NodeJS version
9736daf
Select commit
Loading
Failed to load commit list.
GitHub Actions / Analysis of agora_rtm succeeded Oct 29, 2024 in 0s

Package analysis results for agora_rtm

Score

140/160 points
(87.5%)

Supported platforms

  • Flutter
    • android
    • ios

Details

Follow Dart file conventions (30/30)

✔ 10/10 points: Provide a valid pubspec.yaml

✔ 5/5 points: Provide a valid README.md

✔ 5/5 points: Provide a valid CHANGELOG.md

✔ 10/10 points: Use an OSI-approved license

Detected license: MIT.

Provide documentation (10/20)

⚠ 0/10 points: 20% or more of the public API has dartdoc comments

24 out of 531 API elements (4.5 %) have documentation comments.

Providing good documentation for libraries, classes, functions, and other API elements improves code readability and helps developers find and use your API. Document at least 20% of the public API elements.

To highlight public API members missing documentation consider enabling the public_member_api_docs lint.

Some symbols that are missing documentation: agora_rtm, agora_rtm.AcquireLockResult, agora_rtm.AcquireLockResult.AcquireLockResult, agora_rtm.AcquireLockResult.channelName, agora_rtm.AcquireLockResult.channelType.

✔ 10/10 points: Package has an example

Platform support (20/20)

✔ 20/20 points: Supports 2 of 6 possible platforms (iOS, Android, Web, Windows, macOS, Linux)

  • ✓ Android

  • ✓ iOS

These platforms are not supported:

Package does not support platform `Windows`.

Because:

  • package:agora_rtm/agora_rtm.dart that declares support for platforms: Android, iOS.
Package does not support platform `Linux`.

Because:

  • package:agora_rtm/agora_rtm.dart that declares support for platforms: Android, iOS.
Package does not support platform `macOS`.

Because:

  • package:agora_rtm/agora_rtm.dart that declares support for platforms: Android, iOS.
Package does not support platform `Web`.

Because:

  • package:agora_rtm/agora_rtm.dart that declares support for platforms: Android, iOS.

❌ 0/0 points: WASM compatibility

Package not compatible with runtime wasm

Because:

  • package:agora_rtm/agora_rtm.dart that imports:
  • package:agora_rtm/src/agora_rtm_client_ext.dart that imports:
  • package:agora_rtm/src/impl/agora_rtm_client_impl_override.dart that imports:
  • package:agora_rtm/src/bindings/native_iris_api_engine_binding_delegate.dart that imports:
  • package:iris_method_channel/iris_method_channel_bindings_io.dart that imports:
  • package:iris_method_channel/src/platform/io/bindings/native_iris_api_common_bindings.dart that imports:
  • dart:ffi

This package is not compatible with runtime wasm, and will not be rewarded full points in a future version of the scoring model.

See https://dart.dev/web/wasm for details.

Pass static analysis (40/50)

⚠ 40/50 points: code has no errors, warnings, lints, or formatting issues

Found 12 issues. Showing the first 2:

INFO: 'ignore' is deprecated and shouldn't be used. Use `includeFromJson` and `includeToJson` with a value of `false` instead.

lib/src/agora_rtm_base.dart:1186:36

     ╷
1186 │   @JsonKey(name: 'encryptionSalt', ignore: true)
     │                                    ^^^^^^
     ╵

To reproduce make sure you are using the lints_core and run flutter analyze lib/src/agora_rtm_base.dart

INFO: 'ignore' is deprecated and shouldn't be used. Use `includeFromJson` and `includeToJson` with a value of `false` instead.

lib/src/agora_rtm_client.dart:122:29

    ╷
122 │   @JsonKey(name: 'message', ignore: true)
    │                             ^^^^^^
    ╵

To reproduce make sure you are using the lints_core and run flutter analyze lib/src/agora_rtm_client.dart

Support up-to-date dependencies (40/40)

✔ 10/10 points: All of the package dependencies are supported in the latest version

Package Constraint Compatible Latest
async >=2.8.2 2.12.0 2.12.0
ffi >=1.1.2 2.1.3 2.1.3
flutter flutter 0.0.0 0.0.0
iris_method_channel >=2.2.2 2.2.2 2.2.2
json_annotation ^4.4.0 4.9.0 4.9.0
meta >=1.7.0 1.15.0 1.16.0
Transitive dependencies
Package Constraint Compatible Latest
characters - 1.3.0 1.3.1
collection - 1.18.0 1.19.1
flutter_web_plugins - 0.0.0 0.0.0
js - 0.7.1 0.7.1
material_color_utilities - 0.11.1 0.12.0
sky_engine - 0.0.99 0.0.99
vector_math - 2.1.4 2.1.4

To reproduce run dart pub outdated --no-dev-dependencies --up-to-date --no-dependency-overrides.

✔ 10/10 points: Package supports latest stable Dart and Flutter SDKs

✔ 20/20 points: Compatible with dependency constraint lower bounds

pub downgrade does not expose any static analysis error.

Versions

View more details on GitHub Actions