The GitHub Actions job "Tests (AMD)" on 
airflow.git/feature/dag-schedule-overview-68532 has failed.
Run started by GitHub user anmolxlight (triggered by anmolxlight).

Head commit for run:
a60f1d55f4b684f3e816b3f3306981e212dae703 / anmolxlight <[email protected]>
fix(pr-68547): use optional (?) on formatTooltipBody duration/start/end params

The previous commit added '| undefined' to the duration/start/end types in
formatTooltipBody's parameter, but TypeScript's structural typing is stricter
about optional vs required properties. Since DagScheduleOverviewEntry marks
end_mean_seconds / end_median_seconds / start_mean_seconds / 
start_median_seconds
as '?: number | null' (truly optional, undefined comes from optionality, not
the value type), the parameter must mirror that with '?:' on the same fields.

'?: number | null' and ': number | null | undefined' are NOT structurally
assignable in either direction under TypeScript's strictness here.

Use '?: number | null' to match the auto-generated type shape. The
downstream helpers (formatSecondsAsClock, formatDurationSeconds) still
handle the undefined case and render em-dash.

Local verification:
- pnpm tsc --noEmit -p tsconfig.app.json: clean
- prek run --files <changed file>: Passed
- prek run ts-compile-lint-ui --files <changed file>: Passed

Report URL: https://github.com/apache/airflow/actions/runs/27533084675

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to