deepujain commented on PR #64374:
URL: https://github.com/apache/airflow/pull/64374#issuecomment-4149234441
Validation update after rebasing this PR on `main`:
- I fixed the branch-specific static-check failure in `Header.test.tsx` by
updating the stale DAG mock to match the current `DAGDetailsResponse` shape
after the rebase.
- Local checks rerun and passing:
- `npm run test -- Header.test.tsx`
- `npx eslint src/components/DagDeactivatedBadge.tsx
src/pages/Dag/Header.tsx src/layouts/Details/DagBreadcrumb.tsx
src/layouts/Details/DetailsLayout.tsx src/pages/Dag/Header.test.tsx`
- `pnpm exec tsc --noEmit -p tsconfig.app.json`
- I force-pushed the existing PR branch so CI reruns on the rebased code.
On the last failing run before this push, the browser e2e failures were in
`tests/e2e/specs/home-dashboard.spec.ts` across Chromium / Firefox / WebKit
rather than in the DAG details flow touched here, so they looked unrelated to
this UX change. The fresh run from this push should confirm that with the
current branch state.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
