Skip to content
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.

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

Fix vite example #687

Merged
merged 2 commits into from
Dec 19, 2024
Merged

Fix vite example #687

merged 2 commits into from
Dec 19, 2024

Conversation

jbms
Copy link
Collaborator

@jbms jbms commented Dec 19, 2024

No description provided.

jbms added 2 commits December 18, 2024 21:42
This avoids pulling in unnecessary transitive dependencies, and avoids
issues the vite dev-server has with transitive CommonJS dependencies.
…nstall

The vite examples previously worked only when using a symlinked
Neuroglancer (as in the tests used in this repo), but not when
the Neuroglancer dependency was installed normally.

The vite dev server special-cases symlinked dependencies in a way that
made its optimizeDeps step work when Neuroglancer is symlinked in
`node_modules` but not otherwise.

Additional optimizeDeps options have now been added to the vite example
configs to ensure the build works when Neuroglancer is installed as a
normal dependency.

Fixes #683
@jbms jbms merged commit 3d2f93d into master Dec 19, 2024
26 checks passed
@jbms jbms deleted the fix-vite-example branch December 19, 2024 20:29
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.

1 participant