8000 bump storybook, react, three, yarn, etc by jtbandes · Pull Request #427 · foxglove/three-text · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@jtbandes
Copy link
Member
@jtbandes jtbandes commented Nov 4, 2025

Some things that dependabot was having trouble bumping on its own. Minor tweaking required.

I am hoping the tsconfig change, as well as the THREE.Texture<THREE.DataTextureImageData> type change will not affect downstream consumers?

"outDir": "./dist/cjs",
"module": "commonjs"
"module": "commonjs",
"moduleResolution": "node10"
Copy link

Choose a reason for hiding this comment

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

microsoft/TypeScript#62200 (comment)

changing to esnext and bundler is an option? It seems that it will be deprecated soon, maybe we can already update it

Copy link
Member Author

Choose a reason for hiding this comment

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

This is a separate tsconfig for building as cjs (we build and distribute both cjs + esm for most of our published packages), so we cannot switch to module: esnext. Maybe we could use bundler, not sure if the moduleResolution matters as much, I just kept this to avoid changing too much. I guess we can revisit when TS 6.0 is out?

Copy link
@aneuwald aneuwald left a comment

Choose a reason for hiding this comment

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

just a small question about the node10. But if it gets too much trouble to update it, I guess we can let as it is until we need to bump typescript to v6+

@jtbandes jtbandes merged commit 4e7a73f into main Nov 4, 2025
4 checks passed
@jtbandes jtbandes deleted the jacob/bump-stuff branch November 4, 2025 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0