You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Since #40080 CI fails when running the validate-declarations script. This is caused by ts-node. It is the same issue as mentioned here. The error was not noticed because the old version (18.8.2) was still used for the CI test when the version was upgraded. My idea would therefore be to keep Node.js in version 18.8.2 for the time being until ts-node has a solution ready. Alternatively, you could change the script "ts-node --esm scripts/validateTypescriptDeclarations.mts" to "node --loader ts-node/esm scripts/validateTypescriptDeclarations.mts". However, this uses an experimental feature, so I would tend to use the former.
Thanks for the report and the investigation. I noticed the issue myself today too I would vote for reverting the commit for now too, but I would let @michaldudak check as he is mostly familiar with the infra.
Since #40080 CI fails when running the
validate-declarations
script. This is caused byts-node
. It is the same issue as mentioned here. The error was not noticed because the old version (18.8.2) was still used for the CI test when the version was upgraded. My idea would therefore be to keep Node.js in version 18.8.2 for the time being until ts-node has a solution ready. Alternatively, you could change the script"ts-node --esm scripts/validateTypescriptDeclarations.mts"
to"node --loader ts-node/esm scripts/validateTypescriptDeclarations.mts"
. However, this uses an experimental feature, so I would tend to use the former.Current PRs, where I find the issue:
The text was updated successfully, but these errors were encountered: