Skip to content

Commit

Permalink
fix: use TS_NODE_TYPE_CHECK insteadof TS_NODE_TRANSPILE (#266)
Browse files Browse the repository at this point in the history
<!--
Thank you for your pull request. Please review below requirements.
Bug fixes and new features should include tests and possibly benchmarks.
Contributors guide:
https://github.com/eggjs/egg/blob/master/CONTRIBUTING.md

感谢您贡献代码。请确认下列 checklist 的完成情况。
Bug 修复和新功能必须包含测试,必要时请附上性能测试。
Contributors guide:
https://github.com/eggjs/egg/blob/master/CONTRIBUTING.md
-->

##### Checklist
<!-- Remove items that do not apply. For completed items, change [ ] to
[x]. -->

- [ ] `npm test` passes
- [ ] tests and/or benchmarks are included
- [ ] documentation is changed or added
- [ ] commit message follows commit guidelines

##### Affected core subsystem(s)
<!-- Provide affected core subsystem(s). -->


##### Description of change
<!-- Provide a description of the change below this comment. -->
  • Loading branch information
killagu committed Apr 29, 2024
1 parent 2ee8e88 commit 735cc2f
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions lib/cmd/dal/gen.js
Original file line number Diff line number Diff line change
Expand Up @@ -43,8 +43,9 @@ class DalGenCommand extends Command {
execArgv: context.execArgv,
env: {
...context.env,
// Dal table class modified may cause ts error, so we use transpile only
TS_NODE_TRANSPILE_ONLY: 'true',
// Dal table class modified may cause ts error, so we skip type check
// Don't use TS_NODE_TRANSPILE, it will not ignore typing file.
TS_NODE_TYPE_CHECK: 'false',
},
cwd: baseDir,
};
Expand Down

0 comments on commit 735cc2f

Please sign in to comment.