Skip to content

fix: move typescript to devDependencies - #459

Open
fennoai[bot] wants to merge 1 commit into
masterfrom
fennoai/issue-444-1791892407
Open

fennoai[bot] wants to merge 1 commit into
masterfrom
fennoai/issue-444-1791892407

Conversation

@fennoai

@fennoai fennoai Bot commented Oct 13, 2026

Copy link
Copy Markdown

Requested by @Nliver

将 typescript 从 dependencies 移至 devDependencies,避免使用者安装 SDK 时将 TypeScript 作为生产依赖打包。

验证:npm run check-type、npm pack --dry-run、git diff --check。

@fennoai fennoai Bot left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

No reportable defects were found. The change only reclassifies typescript as a development dependency; repository searches found no runtime usage, and CI/publish workflows continue to install development dependencies.

Reviewed head commit: fa1cc7cd8ba3b34bed02f5ba20dafd78ac94fb46

@fennoai

fennoai Bot commented Oct 13, 2026 •

Copy link
Copy Markdown
Author

审查结论

未发现由本 PR 引入的可操作问题,补丁整体正确。

已验证

  • typescript 已从 dependencies 移至 devDependencies。
  • 干净安装开发依赖后,npm run check-type 通过。
  • npm pack --dry-run 通过,TypeScript 未被打包进发布内容。
  • 生产模式 npm install --omit=dev 后可正常 require('qiniu'),且生产依赖树不包含 TypeScript。
  • 固定提交范围执行 git diff --check 通过。

未运行完整测试套件;本次变更仅涉及依赖分类。

View job run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant