dart-lang / native / 12670761014

88%

Build Type

push

github

Committed by web-flow

Commit Message

[native_assets_cli] Refactor into `BuildInput` (#1871)

This PR changes the public API according to https://github.com/dart-lang/native/issues/1738, and should have minimal behavior changes.

Everything in `HookConfig` causes a different `input.outputDirectory`, all the other elements in the `input` do not.

* This leads to the `packageName` no longer influencing the checksum, so the default output directory has been changed to `.dart_tool/native_assets_builder/$packageName/$checksum`.

Follow up PRs:

* Serialize to a new JSON format
* get rid of cCompiler envScript

Design:

* https://github.com/dart-lang/native/issues/1738

Run Details

218 of 272 new or added lines in 15 files covered. (80.15%)

1 existing line in 1 file now uncovered.

12283 of 13975 relevant lines covered (87.89%)

33.95 hits per line