增加ts的支持 #15
增加ts的支持 #15
Conversation
Codecov Report
@@ Coverage Diff @@
## master #15 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 8 7 -1
Lines 94 92 -2
=====================================
- Hits 94 92 -2
Continue to review full report at Codecov.
|
README.md
Outdated
| 在项目根目录创建 tshelper.js,并赋值如下代码: | ||
|
|
||
| ``` | ||
| module.exports = require("egg-graphql/tshelper"); |
There was a problem hiding this comment.
是的，看到egg-ts-helper源码中定义了默加载路径，只是提供了对外的配置，没有想到pull-request
tshelper.js
Outdated
| const generator = require('./lib/ts-generator/connector'); | ||
|
|
||
| module.exports = { | ||
| watchDirs: { |
There was a problem hiding this comment.
直接提 PR 到 egg-ts-helper 中吧，不要把生成 d.ts 的功能集成到插件里
There was a problem hiding this comment.
@whxaxes 最近遇到个问题，我自定义的ts文件中的declare module 'egg' {}会覆盖掉egg的index.d.ts,而egg-ts-helper自定义生成的文件却可以合并。请问是怎么合并的？
There was a problem hiding this comment.
你在 declare module 'egg' 前面加个 import ，import 什么都行，然后就能正常合并了......
|
什么情况 ，为什么不合并呢 |
|
@whxaxes 这个 PR 进度如何？ @ShuaiShuaiguo 需要 Rebase 下 |
|
有进展吗？ |
|
没人维护了吗？进度就这么停着？ |
|
进展如何？ |
|
看底部，需要 rebase 处理冲突问题。 |
|
@ShuaiShuaiguo 方便处理下冲突吗？想升级到 egg-bin + egg-ts-helper 的方式，egg-graphql 加载的是 js 文件，导致 egg-bin dev 开发时 app/graphql 文件夹对应 js 文件不会自动生成 |
Codecov Report
@@ Coverage Diff @@
## master #15 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 8 7 -1
Lines 110 92 -18
=====================================
- Hits 110 92 -18
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #15 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 8 7 -1
Lines 110 102 -8
=====================================
- Hits 110 102 -8
Continue to review full report at Codecov.
|
|
非常抱歉，由于工作原因且graphql项目始终无法落地，故关注度降低。egg-ts-helper支持请参照，希望提PR |
|
@ShuaiShuaiguo 感谢，commits 可能需要 rebase 合并下，egg-ts-helper 后面我可以做下支持 |
|
自己整合了下代码。。。着急的童鞋可以用这个版本： |
|
大佬们，啥时候 ts 支持能落地啊 ... |
Checklist
npm testpassesAffected core subsystem(s)
Description of change