豆豆友情提示:这是一个非官方 GitHub 代理镜像,主要用于网络测试或访问加速。请勿在此进行登录、注册或处理任何敏感信息。进行这些操作请务必访问官方网站 github.com。 Raw 内容也通过此代理提供。
Skip to content

chore: Remove file unused after bundling from published package#785

Merged
zyzyzyryxy merged 1 commit intomainfrom
chore-remove-bundled-devtools-js-file
Jan 19, 2026
Merged

chore: Remove file unused after bundling from published package#785
zyzyzyryxy merged 1 commit intomainfrom
chore-remove-bundled-devtools-js-file

Conversation

@zyzyzyryxy
Copy link
Copy Markdown
Contributor

build/src/third_party/devtools.js is only used during bundling, but we publish it unnecessarily. We should remove it to not cause confusion and slightly decrease package size.

@zyzyzyryxy zyzyzyryxy requested a review from OrKoN January 16, 2026 16:55
@zyzyzyryxy zyzyzyryxy changed the title chore: remove file unused after bundling from published package chore: Remove file unused after bundling from published package Jan 16, 2026
@zyzyzyryxy zyzyzyryxy force-pushed the chore-remove-bundled-devtools-js-file branch from e5a6fa8 to f71fcc1 Compare January 16, 2026 17:05
Copy link
Copy Markdown
Collaborator

@OrKoN OrKoN left a comment

Choose a reason for hiding this comment

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

Thanks! instead of this, let's move re-exports from devtools.ts into the index.ts and delete devtools.ts

@zyzyzyryxy zyzyzyryxy closed this Jan 19, 2026
@zyzyzyryxy zyzyzyryxy force-pushed the chore-remove-bundled-devtools-js-file branch from f71fcc1 to 7ba63a8 Compare January 19, 2026 10:41
@zyzyzyryxy zyzyzyryxy reopened this Jan 19, 2026
@zyzyzyryxy zyzyzyryxy enabled auto-merge January 19, 2026 11:39
@zyzyzyryxy zyzyzyryxy added this pull request to the merge queue Jan 19, 2026
Merged via the queue into main with commit 3c8ce2d Jan 19, 2026
43 of 49 checks passed
@zyzyzyryxy zyzyzyryxy deleted the chore-remove-bundled-devtools-js-file branch January 19, 2026 11:48
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