From 735cc2f1c7eb8ce84e324903bd85ee01961fbaba Mon Sep 17 00:00:00 2001
From: killa <killa123@126.com>
Date: Mon, 29 Apr 2024 19:17:29 +0800
Subject: [PATCH] fix: use TS_NODE_TYPE_CHECK insteadof TS_NODE_TRANSPILE
 (#266)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

<!--
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. -->
---
 lib/cmd/dal/gen.js | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/lib/cmd/dal/gen.js b/lib/cmd/dal/gen.js
index 2b54ed6..4ef4418 100644
--- a/lib/cmd/dal/gen.js
+++ b/lib/cmd/dal/gen.js
@@ -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,
     };