dart-lang / native / 14077318286 / 6 – native_assets_builder_ubuntu
81%
main: 81%

DEFAULT BRANCH: main
Ran
Files 11
Run time 0s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 89.562%. Remained the same
native_assets_builder_ubuntu – 14077318286.6

push

github

web-flow 
[native_assets_cli] Fix snake casing in JSON (#2128)

Closes: https://github.com/dart-lang/native/issues/2037
Closes: https://github.com/dart-lang/native/issues/2038

Fixes spelling in the JSON format:

* `prefer-dynamic` -> `prefer_dynamic`
* `prefer-static` -> `prefer_static`
* `assetsForLinking` -> `assets_for_linking`

### PR changes

* `pkgs/hook`, `pkgs/code_assets`, and `pkgs/data_assets`
  * Introduces new schemas in the non-generated `.schema.json` files
  * Updates the generated schemas to pick up on the new spelling for `assets_for_linking`.
  * Adds test data for the new spelling.
  * Added JSON schema tests for the new spelling.

* `package:native_assets_cli`
  * Generated the syntax classes
  * Deal with version skew in the mapping between syntax classes and semantic API

### Version skew between hooks and SDKs

* For https://github.com/dart-lang/native/issues/2037
  * Backwards compatibility older SDKs: Also still emit `assetsForLinking`, instead of only `assets_for_linking`.
  * Backwards compatibility older hooks: Also still read `assetsForLinking`, instead of only `assets_for_linking`.

* For https://github.com/dart-lang/native/issues/2038
  * Backwards compatibility older SDKs: Still read the casing with dashes.
  * Backwards compatibility older hooks: Irrelevant. Both Flutter and Dart only use `dynamic`.

695 of 776 relevant lines covered (89.56%)

41.17 hits per line

Source Files on job native_assets_builder_ubuntu - 14077318286.6