Conversation
There was a problem hiding this comment.
よくわからないけどこのファイルの変更は不要っぽいかも
ちょっとAI君に修正してもらってみます！！
There was a problem hiding this comment.
@copilot宛にコメント書こうとしたらwarningが。。
copilot君の権限的に、他の人のPRへ僕からメンションはできないのかな。しゃーない！
There was a problem hiding this comment.
あ〜多分prettier -w .をかけた時のやつですね…
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the module loading strategy in NodeDictionaryLoader to use dynamic imports instead of static imports, and updates the NOTICE.md file formatting to use proper Markdown syntax.
- Replaced static imports of
fsandzlibwith a dynamic module loading system usingimport() - Introduced a caching mechanism for dynamically loaded modules
- Reformatted NOTICE.md to use standard Markdown headings and list syntax
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| src/loader/NodeDictionaryLoader.js | Refactored to use dynamic imports with caching instead of static imports for fs and zlib modules |
| NOTICE.md | Updated formatting to use proper Markdown syntax for headings, lists, and improved spacing |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| }); | ||
| }); | ||
| }, | ||
| ); |
There was a problem hiding this comment.
The Promise rejection is not handled. If either module fails to load, the callback will never be called, leaving the operation hanging. Add a .catch() handler that calls callback(err).
| ); | |
| ).catch((err) => callback(err)); |
There was a problem hiding this comment.
copilot君のレビュー、ほかはともかくこれは導入しても良いのかもしれない
まあでもimportエラーだけは実行時エラーでも良い気もしないでもない。
以前はそうだったはずだろうし。
んーー名無し。さんてきにどっちでも良さそうだったらスルーで良さそう！
Hiroshiba
left a comment
Hiroshiba
left a comment
There was a problem hiding this comment.
LGTM！！
copilot君のレビューでなんか良さそうなのあったら適用しても良さそう。
んでマージして頂ければ！！
| }); | ||
| }); | ||
| }, | ||
| ); |
There was a problem hiding this comment.
copilot君のレビュー、ほかはともかくこれは導入しても良いのかもしれない
まあでもimportエラーだけは実行時エラーでも良い気もしないでもない。
以前はそうだったはずだろうし。
んーー名無し。さんてきにどっちでも良さそうだったらスルーで良さそう！
|
マージ権限がないっぽいです。 |
|
あそうか、preview-pages入ってないんだった。 |
2 participants
内容
動的インポートでViteの警告が出ないようにします。
関連 Issue
（なし）
スクリーンショット・動画など
（なし）
その他
（なし）