Skip to content

[v3-2-test] Upgrade to Vite 8. (#65084)#65166

Merged
potiuk merged 1 commit into
v3-2-testfrom
backport-bcd31b2-v3-2-test
Apr 27, 2026
Merged

[v3-2-test] Upgrade to Vite 8. (#65084)#65166
potiuk merged 1 commit into
v3-2-testfrom
backport-bcd31b2-v3-2-test

Conversation

@tirkarthi
Copy link
Copy Markdown
Contributor

  • Upgrade to Vite 8.

  • Update vite.config.ts for plugin-react@6.0.0.

cherry picked from commit bcd31b2

Was generative AI tooling used to co-author this PR?

No

@boring-cyborg boring-cyborg Bot added the area:UI Related to UI/UX. For Frontend Developers. label Apr 13, 2026
@tirkarthi tirkarthi mentioned this pull request Apr 13, 2026
Copy link
Copy Markdown
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good but static checks need fixing.

@tirkarthi tirkarthi force-pushed the backport-bcd31b2-v3-2-test branch from 6273bcd to 9f225da Compare April 14, 2026 16:06
@tirkarthi
Copy link
Copy Markdown
Contributor Author

Looks like msw is upgraded to 2.13.x from 2.12.x which could be causing the error. This was not upgraded in main during vite 8 upgrade so not sure why backport tries to install 2.13.x .

Copy link
Copy Markdown
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I reran the failing tests, but I think we're getting somewhere.

Can we clean the branch please (base branch was rebased hence the weird looking diff)

@potiuk potiuk force-pushed the backport-bcd31b2-v3-2-test branch from 653eff8 to 97fd2dc Compare April 26, 2026 23:51
@potiuk potiuk merged commit c0fd633 into v3-2-test Apr 27, 2026
79 checks passed
@potiuk potiuk deleted the backport-bcd31b2-v3-2-test branch April 27, 2026 01:02
potiuk added a commit that referenced this pull request Apr 27, 2026
The dependabot bump in this PR brings in newer typescript-eslint and
msw versions that produce TS errors and lint diffs against the
existing v3-2-test code:

* eslint with @typescript-eslint v8 flags several `as X` casts as
  redundant — auto-fix removes them, and two of the now-unused symbols
  (`ColorMode` type alias, `ParamsSpec` import) need follow-up cleanup.
* The msw `SetupServerApi` type lost the `#private` and `network`
  members; the existing `let server: SetupServerApi` declaration no
  longer accepts the value returned by `setupServer(...)`. Same fix as
  PR #65166 — drop the annotation, let TypeScript infer.
vatsrahul1001 pushed a commit that referenced this pull request Apr 27, 2026
The dependabot bump in this PR brings in newer typescript-eslint and
msw versions that produce TS errors and lint diffs against the
existing v3-2-test code:

* eslint with @typescript-eslint v8 flags several `as X` casts as
  redundant — auto-fix removes them, and two of the now-unused symbols
  (`ColorMode` type alias, `ParamsSpec` import) need follow-up cleanup.
* The msw `SetupServerApi` type lost the `#private` and `network`
  members; the existing `let server: SetupServerApi` declaration no
  longer accepts the value returned by `setupServer(...)`. Same fix as
  PR #65166 — drop the annotation, let TypeScript infer.
@tirkarthi
Copy link
Copy Markdown
Contributor Author

Thanks @potiuk

@vatsrahul1001 vatsrahul1001 added this to the Airflow 3.2.2 milestone May 13, 2026
@vatsrahul1001 vatsrahul1001 added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label May 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:UI Related to UI/UX. For Frontend Developers. changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants