Skip to content

chore(deps): downgrade vite to v7.2.9 #39930

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
jamietanna merged 1 commit into from
Dec 12, 2025
Merged

chore(deps): downgrade vite to v7.2.9 #39930

jamietanna merged 1 commit into from
Dec 12, 2025

Conversation

@jamietanna
Copy link
Contributor

@jamietanna jamietanna commented Dec 12, 2025

Changes

Since the upgrade yesterday in #39891, we've seen a bit more flakiness
in our CI pipeline, as noted in #39921.

Although we've not confirmed if this is the issue, it appears to be
suspicious timing.

Out of an abundance of caution, we'll revert the upgrade to see if this
resolves the issue.

This reverts #39891.

Context

Please select one of the below:

  • This closes an existing Issue: #
  • This doesn't close an Issue, but I accept the risk that this PR may be closed if maintainers disagree with its opening or implementation

AI assistance disclosure

Did you use AI tools to create any part of this pull request?

Please select one option and, if yes, briefly describe how AI was used (e.g., code, tests, docs) and which tool(s) you used.

  • No — I did not use AI for this contribution.
  • Yes — minimal assistance (e.g., IDE autocomplete, small code completions, grammar fixes).
  • Yes — substantive assistance (AI generated non‑trivial portions of code, tests, or documentation).
  • Yes — other (please describe):

Documentation (please check one with an [x])

  • I have updated the documentation, or
  • No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

  • Code inspection only, or
  • Newly added/modified unit tests, or
  • No unit tests but ran on a real repository, or
  • Both unit tests + ran on a real repository

The public repository:

@jamietanna
chore(deps): downgrade vite to v7.2.9
2904c99 
Since the upgrade yesterday in #39891, we've seen a bit more flakiness
in our CI pipeline, as noted in #39921.

Although we've not confirmed if this is the issue, it appears to be
suspicious timing.

Out of an abundance of caution, we'll revert the upgrade to see if this
resolves the issue.

This reverts commit 58f748f.
@jamietanna jamietanna requested a review from viceice December 12, 2025 08:53
@jamietanna
Copy link
Contributor Author

jamietanna commented Dec 12, 2025 •
edited
Loading

I'm going to run CI several times on this branch to get some more data about whether it does fix this

@jamietanna
Copy link
Contributor Author

jamietanna commented Dec 12, 2025 •
edited
Loading

3 runs so far have been all green, 4th pending then will merge

@jamietanna jamietanna added this pull request to the merge queue Dec 12, 2025
@jamietanna jamietanna mentioned this pull request Dec 12, 2025
Merged
12 tasks
Merged via the queue into main with commit 5977653 Dec 12, 2025
137 checks passed