2 – native_toolchain_c_windows
74%
main: 74%

Ran 28 Apr 2025 01:34PM UTC

Files 26

Run time 0s

Badge

Embed ▾

Committed 28 Apr 2025 01:25PM UTC coverage: 79.181%. Remained the same

Subproject # native_toolchain_c_windows – 14708923181.2

Build Type

push

github

Committed by

web-flow

Commit Message

[native_assets_cli] Replace extension types with classes (#2251)

This PR replaces all extension types with classes.

This PR renames all classes that mostly belong to another class to start with the name of that class.

I'm not entirely satisfied with how this looks, but I think it's a step in the right direction.

The Dart docs can be inspected by `pkgs/native_assets_cli$ dart doc .`.

Open question:

* Should we _not_ export all those helper classes? That would prevent users using these types in their code. But this would be considered a leaked symbol. So this probably doesn't work.
* Should we not generate dartdocs for those helper classes. They usually simply have an `add`, `addAll` or `operator []`. The interesting doc comments are on the accessors that return those classes.

Context:

* Work before splitting up the package: https://github.com/dart-lang/native/issues/999

Run Details

890 of 1124 relevant lines covered (79.18%)

5.25 hits per line

dart-lang / native / 14708923181 / 2 – native_toolchain_c_windows
74%
main: 74%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job native_toolchain_c_windows - 14708923181.2

dart-lang / native / 14708923181 / 2 – native_toolchain_c_windows 74% main: 74%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job native_toolchain_c_windows - 14708923181.2

dart-lang / native / 14708923181 / 2 – native_toolchain_c_windows
74%
main: 74%

README BADGES
x