think-template / typescript

ThinkJS typescript cli template
MIT License
6 stars 5 forks source link

[question] 这里为什么是require和import混用的呢? #3

Open muzea opened 5 years ago

muzea commented 5 years ago

https://github.com/think-template/typescript/blob/5958f138253dc0275483369edab6951fb57aaf82/template/src/config/extend.ts#L1-L5

按道理是可以全部使用 import 的吧?

lizheming commented 5 years ago

是的,应该是从 JS 脚手架迁移的时候忘记替换了,欢迎 PR

muzea commented 5 years ago

看上去需要点时间

错误的类型导出

这里的类型导出一个 namespace 是为什么呢?有什么地方会依赖导出类型吗?

缺少类型定义

lizheming commented 5 years ago

一个模块的 dts 定义一个 namespace 不是很正常?

muzea commented 5 years ago

那也不能光写一个 namespace 。。。 应该和 moment 类似,得导出一个函数或者对象。

lizheming commented 5 years ago

嗯,可以导出,不过由于这些都是 ThinkJS 的模块,很少有单独使用的,最后都需要挂载到 ThinkJS 的对象上,所以我们的主要目的是利用 interface 的自动合并声明到 thinkjs 的 namespace 中。因为安装模块后不是按照正常思路的 require('think-session').session 而是 think.session

muzea commented 5 years ago

那需要补的类型定义有点小多,估计半个月后才会有时间补了 :(

lizheming commented 5 years ago

OK,没事,有空就帮忙改下就行,先给你点个赞,233