8 – native_assets_builder_windows
58%
main: 58%

Ran 06 Dec 2024 12:00PM UTC

Files 11

Run time 0s

Badge

Embed ▾

Committed 06 Dec 2024 11:52AM UTC coverage: 90.922% (-0.1%) from 91.027%

Subproject # native_assets_builder_windows – 12198315502.8

Build Type

push

github

Committed by

web-flow

Commit Message

[native_assets_builder] Report dart sources as dependencies of hooks (#1780)

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

This PR adds the Dart sources to the dependencies in `HookResult`.

It also fixes the dep-file parsing w.r.t. to escapes.

### Implementation details

We're passing around `HookOutput`s which is the deserialized `output.json`. We could add the Dart sources as dependencies to it _after_ deserializing, but then the correspondence to the json is lost. So instead I've added an extra return value to the places where we pass `HookOutput` around.

Coverage Stats

651 of 716 relevant lines covered (90.92%)

29.28 hits per line

dart-lang / native / 12198315502 / 8 – native_assets_builder_windows
58%
main: 58%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job native_assets_builder_windows - 12198315502.8

dart-lang / native / 12198315502 / 8 – native_assets_builder_windows 58% main: 58%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job native_assets_builder_windows - 12198315502.8

dart-lang / native / 12198315502 / 8 – native_assets_builder_windows
58%
main: 58%

README BADGES
x