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

chore(deps): update globby #141

Closed
wants to merge 2 commits into from
Closed

Conversation

waitingsong
Copy link
Contributor

Checklist
  • npm test passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)
Description of change

update globby to ^10.0.0

@codecov
Copy link

codecov bot commented Oct 9, 2019

Codecov Report

Merging #141 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #141   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          12     12           
  Lines         229    229           
=====================================
  Hits          229    229

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9cb8125...c9a5633. Read the comment docs.

@atian25
Copy link
Member

atian25 commented Oct 9, 2019

我们之前一般都执行下 egg-bin autod,批量升级,然后逐个变更点去看它的 changelog,特别是大版本的都要看下。

@waitingsong
Copy link
Contributor Author

waitingsong commented Oct 9, 2019

globby 这个依赖, egg /midway 各个包里面版本各有不同。结果我项目里面安装了多个版本。所以干脆直接用 midway 里面的最高版本来单独升级这个依赖。

另外, node v6 已经不维护, v8 也将于今年年底停止支持。 eggjs 也应该删除 v6 版本的支持了吧。

@atian25
Copy link
Member

atian25 commented Oct 9, 2019

egg 2 里面早已经移除了 6 了。

就只剩 egg-bin 几个 cli 还没升级,回头直接升级到 10 就好了,还有几个月。

.travis.yml Outdated Show resolved Hide resolved
@waitingsong
Copy link
Contributor Author

nodejs v6 依然失败,其它没问题。

@atian25
Copy link
Member

atian25 commented Oct 12, 2019

globby 10 不支持 node 6,那这个 PR 暂时 hold 下吧?

@waitingsong
Copy link
Contributor Author

@atian25 那关掉 PR 等 egg 移除 v6 支持再说?

@atian25
Copy link
Member

atian25 commented Oct 12, 2019

先放着不动吧,bin 可以先不升,其他 egg 插件可以升,只要它已经是 8+ 的

@waitingsong
Copy link
Contributor Author

另外,ts-node, typescript 的依赖相差一个大版本,是否考虑升上去呢。

@whxaxes
Copy link
Member

whxaxes commented Oct 12, 2019

@waitingsong ts-node 8.x 编译速度巨慢,之前升过一次,又回滚回去了

@whxaxes
Copy link
Member

whxaxes commented Oct 12, 2019

typescript 可以升到 3

@waitingsong
Copy link
Contributor Author

egg-bin 是 js 项目,只是在单测里面才用了 ts。就几个用例啊……

@whxaxes
Copy link
Member

whxaxes commented Oct 12, 2019

@waitingsong ts-node 不是仅用于 egg-bin 的,是用于 egg 应用的启动的

@waitingsong
Copy link
Contributor Author

那就先升 ts: #144

@waitingsong
Copy link
Contributor Author

@whxaxes 这个 tsc 导致的启动慢具体在跑 CI 慢,还是实际生产环境启动慢呢

@atian25
Copy link
Member

atian25 commented Oct 12, 2019

是 ts-node 导致启动慢,不是 tsc。

前者只是本地开发环境才用的

@whxaxes
Copy link
Member

whxaxes commented Oct 12, 2019

@waitingsong
Copy link
Contributor Author

@atian25 哦,记混淆了。 tsc 是编译用途, ts-node 可以直接运行 .ts 文件。

另外,根据 ts 官方那个链接,貌似开启 incremental 功能后性能有所提高。

@whxaxes
Copy link
Member

whxaxes commented Oct 12, 2019

incremental 是只适用于 tsc 的,ts-node 不支持

@fengmk2 fengmk2 closed this Jun 3, 2022
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.

5 participants