Skip to content

fix: Combobox exported type error#2351

Merged
mofojed merged 1 commit intodeephaven:mainfrom
mofojed:fix-combobox-type-export
Jan 28, 2025
Merged

fix: Combobox exported type error#2351
mofojed merged 1 commit intodeephaven:mainfrom
mofojed:fix-combobox-type-export

Conversation

@mofojed
Copy link
Copy Markdown
Member

@mofojed mofojed commented Jan 28, 2025

- Was receiving an error on running `npm start`: [ERROR] Failed to resolve entry for package "@react-types/combobox". The package may have incorrect main/module/exports specified in its package.json. [plugin vite:dep-scan]
- Seemed to be an issue with how we exported the type. An issue has been filed with esbuild: evanw/esbuild#4054
- Just change the export so that we avoid this issue. Tested by running `npm start` again
@mofojed mofojed requested a review from bmingles January 28, 2025 22:04
@mofojed mofojed self-assigned this Jan 28, 2025
Copy link
Copy Markdown
Contributor

@bmingles bmingles left a comment

Choose a reason for hiding this comment

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

Code looks good to me.

Code coverage failed with the known intermittent error so may need to kick that off again.

@mofojed mofojed enabled auto-merge (squash) January 28, 2025 22:11
@mofojed mofojed merged commit 40dfcb7 into deephaven:main Jan 28, 2025
@mofojed mofojed deleted the fix-combobox-type-export branch January 28, 2025 22:19
@github-actions github-actions Bot locked and limited conversation to collaborators Jan 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants