Skip to content

Vite v8 upgrade#292

Merged
devmount merged 3 commits into
mainfrom
chores/vite8-upgrade
Jun 16, 2026
Merged

Vite v8 upgrade#292
devmount merged 3 commits into
mainfrom
chores/vite8-upgrade

Conversation

@devmount

@devmount devmount commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

What changed?

After having issues in #274 with peer dependencies, we decided to break that change down. This PR only does the Vite 8 upgrade and fixes 3 type mismatches reported when building the project. Also a minor Storybook update was applied.

Limitations and Notes

None

Applicable Issues

Related to #273

@davinotdavid davinotdavid left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM after updating storybook with v10.4.6

Comment thread package.json Outdated
"prettier": "3.7.4",
"react": "19.2.4",
"storybook": "10.3.5",
"storybook": "10.4.5",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

NPM packages are updated like crazy recently 😅 There was another update to Storybook 6 hours ago that would be nice to add here:

https://github.com/storybookjs/storybook/releases/tag/v10.4.6

This apparently updates the esbuild version that has a current vulnerability.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Haha indeed! That was shortly after me creating that PR 😂 I'll update it, thanks!

@devmount devmount merged commit ce6c69b into main Jun 16, 2026
3 checks passed
@devmount devmount deleted the chores/vite8-upgrade branch June 16, 2026 18:06
@devmount devmount changed the title Vite8 upgrade Vite v8 upgrade Jun 16, 2026
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.

2 participants