-
Notifications
You must be signed in to change notification settings - Fork 218
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Imprv/156177 159580 change pdf converter type to module #9520
Imprv/156177 159580 change pdf converter type to module #9520
Conversation
|
"start:prod:ci": "pnpm start:prod --ci", | ||
"start:prod": "node dist/index.js", | ||
"lint": "pnpm eslint **/*.{js,ts}", | ||
"gen:swagger-spec": "SWAGGER_GENERATION=true tsed run generate-swagger --output ./specs", | ||
"gen:swagger-spec": "SWAGGER_GENERATION=true node --import @swc-node/register/esm-register src/bin/index.ts generate-swagger --output ./specs", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tsed run だとバグがあり、エラーが生じるようになってしまったが、内部実装を確認するとただのエイリアスコマンドであり、実行している内容は node --import @swc-node/register/esm-register src/bin/index.ts
だったため、直接そちらを書くように変更。
@tsed/cli は依存するパッケージの量も多いため、generate-swagger のみを実行する場合、こちらの方が軽くて良さげ。
"ignore": ["node_modules/**/*"], | ||
"delay": 100, | ||
"execMap": { | ||
"ts": "node --import @swc-node/register/esm-register" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dev での ts の実行に swc-node を使うように変更
- ts-node では esm の実行でエラーが生じる問題が解決していない: TypeError [ERR_UNKNOWN_FILE_EXTENSION]: Unknown file extension ".ts" in Node v21.5.0 TypeStrong/ts-node#2100
- ts-node の alternative としてよく使用されている tsx は、ts.ed で必須の emitDecoratorMetadata が有効にできないという問題が解決していない: Support
emitDecoratorMetadata
privatenumber/tsx#347
…580-change-pdf-converter-type-to-module
edea8be
into
feat/135772-pdf-page-bulk-export
https://github.com/weseek/growi/pull/9279/files#r1813397949 対応
実装内容
動作確認
pdf エクスポートができることを確認
task
https://redmine.weseek.co.jp/issues/159580