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.

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

Changed path dependencies to use pub.dev and API compatible version ranges #55

Closed
wants to merge 1 commit into from

Changed path dependencies to use pub.dev and API compatible version r…

2b263f6
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Closed

Changed path dependencies to use pub.dev and API compatible version ranges #55

Changed path dependencies to use pub.dev and API compatible version r…
2b263f6
Select commit
Loading
Failed to load commit list.
GitHub Actions / Analysis of microsoft_kiota_abstractions succeeded Dec 5, 2024 in 38s

Package analysis results for microsoft_kiota_abstractions

Score

130/160 points
(81.25%)

Supported platforms

  • Flutter
    • android
    • ios
    • windows
    • linux
    • macos
    • web
  • Dart
    • native-aot
      Can be aot compiled with eg. dart2native (Can use dart:io but not dart:mirrors)
    • native-jit
      Can be run with the dart vm in jit mode. (Can use dart:io and dart:mirrors)
    • web
      Can be compiled with DDC and dart2js. (Can use dart:html and friends, not dart:io, dart:mirrors, dart:ffi, etc.)

Details

Follow Dart file conventions (20/30)

❌ 0/10 points: Provide a valid pubspec.yaml

Failed to verify repository URL.

Please provide a valid repository URL in pubspec.yaml, such that:

  • repository can be cloned,
  • a clone of the repository contains a pubspec.yaml, which:,
    • contains name: microsoft_kiota_abstractions,
    • contains a version property, and,
    • does not contain a publish_to property.

packages/microsoft_kiota_abstractions/pubspec.yaml from the repository defines publish_to, thus, we are unable to verify the package is published from here.

✔ 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)

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

272 out of 306 API elements (88.9 %) have documentation comments.

Some symbols that are missing documentation: microsoft_kiota_abstractions.ApiClientBuilder.enableBackingStoreForParseNodeFactory, microsoft_kiota_abstractions.ApiClientBuilder.enableBackingStoreForSerializationWriterFactory, microsoft_kiota_abstractions.ApiClientBuilder.registerDefaultDeserializer, microsoft_kiota_abstractions.ApiClientBuilder.registerDefaultSerializer, microsoft_kiota_abstractions.BaseRequestBuilderExtensions.

❌ 0/10 points: Package has an example

No example found.

See package layout guidelines on how to add an example.

Platform support (20/20)

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

  • ✓ Android

  • ✓ iOS

  • ✓ Windows

  • ✓ Linux

  • ✓ macOS

  • ✓ Web

✔ 0/0 points: WASM compatibility

This package is compatible with runtime wasm, and will be rewarded additional 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

INFO: The null check operator shouldn't be used on a variable whose type is a potentially nullable type parameter.

lib/src/case_insensitive_map.dart:136:51

    ╷
136 │       if (test(_originalKeys[key]!, _contents[key]!)) {
    │                                                   ^
    ╵

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

INFO: The null check operator shouldn't be used on a variable whose type is a potentially nullable type parameter.

lib/src/case_insensitive_map.dart:157:66

    ╷
157 │       _contents[key] = update(_originalKeys[key]!, _contents[key]!);
    │                                                                  ^
    ╵

To reproduce make sure you are using the lints_core and run dart analyze lib/src/case_insensitive_map.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
meta ^1.16.0 1.16.0 1.16.0
std_uritemplate ^2.0.1 2.0.1 2.0.1
uuid ^4.5.1 4.5.1 4.5.1
Transitive dependencies
Package Constraint Compatible Latest
collection - 1.19.1 1.19.1
crypto - 3.0.6 3.0.6
fixnum - 1.1.1 1.1.1
sprintf - 7.0.0 7.0.0
typed_data - 1.4.0 1.4.0

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