Skip to content

build(deps): replace rolldown-vite@^7 with vite@^8#529

Open
cwillisf wants to merge 1 commit intodevelopfrom
vite-8
Open

build(deps): replace rolldown-vite@^7 with vite@^8#529
cwillisf wants to merge 1 commit intodevelopfrom
vite-8

Conversation

@cwillisf
Copy link
Copy Markdown
Contributor

@cwillisf cwillisf commented Apr 15, 2026

Proposed Changes

Replace rolldown-vite@^7 with vite@^8

Reason for Changes

Vite 8 has been released, and defaults to using rolldown.

Test Coverage

Covered by existing tests.

@cwillisf cwillisf requested a review from Copilot April 15, 2026 00:20
@cwillisf cwillisf requested a review from a team as a code owner April 15, 2026 00:20
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the @scratch/task-herder workspace to use official Vite v8 now that Vite defaults to Rolldown, removing the prior npm alias override to rolldown-vite.

Changes:

  • Remove the overrides entry that aliased vite to npm:rolldown-vite@7.3.1.
  • Set packages/task-herder devDependency vite to 8.0.8.
  • Regenerate package-lock.json to reflect the new Vite/Rolldown dependency graph.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
packages/task-herder/package.json Removes the vite alias override and pins Vite to 8.0.8 for the task-herder workspace.
package-lock.json Updates lockfile entries from the Vite/Rolldown toolchain change (including Vite 8.0.8 and Rolldown rc.15).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Copy Markdown
Contributor

Test Results

    8 files  1 083 suites   12m 6s ⏱️
2 720 tests 2 712 ✅  8 💤 0 ❌
6 273 runs  6 234 ✅ 39 💤 0 ❌

Results for commit f54643d.

@github-actions
Copy link
Copy Markdown
Contributor

Test Results

    8 files  1 083 suites   12m 5s ⏱️
2 720 tests 2 712 ✅  8 💤 0 ❌
6 273 runs  6 234 ✅ 39 💤 0 ❌

Results for commit f54643d.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants