dart-lang / native / 8341450886

92%

push

github

[native_assets_builder] `build.dart` -> `hook/build.dart` (#1018)

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

Notes:

* Should keep existing toplevel `build.dart` scripts working by trying both `hook/build.dart` and `build.dart`.
* Does not migrate the examples yet, so that we don't have to do `breaking-change` is true in the ci.
   * Will do this after rolling into Dart.
* `native_toolchain_c` had a `this.dartBuildFiles = const ['build.dart'],`. Updating that to `hook/build.dart` would not work for unmigrated scripts, because files listed as dependencies that don't exist will also trigger a rebuild, e.g. it is considered deleting a file. It would also break on any kind of invocation from `hook/link.dart`. So, I've removed the default value for now. It's not pretty though. Ideas @mosuem?

10 of 11 new or added lines in 2 files covered. (90.91%)

5848 of 6245 relevant lines covered (93.64%)

24.26 hits per line