Skip to content
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

リファクタリング:fsとpathとosとdotenvは名前付きインポートを使わずにデフォルトインポートを使うようにする #2608

Open
Hiroshiba opened this issue Mar 9, 2025 · 0 comments
Labels
初心者歓迎タスク 初心者にも優しい簡単めなタスク 機能向上

Comments

@Hiroshiba
Copy link
Member

内容

名前付きインポート↓

import { rm } from "fs/promises";

ではなくデフォルトインポート↓

import fs from "fs/promises";

を使っていきたい、というissueです。

特にpath.resolvePromise.resolveはどちらも使うし結構危ない気がします。
あとpathのjoinとかosのarchとかも。

Pros 良くなる点

表記揺れが少なくなってコードの再利用性が高まる。

実現方法

とりあえずなんとなく、fspathosdotenvを対象にするので良いかなーと思いました。
vueのwatchとかも一般名詞で危ない気もしますが、まあ流石に意識できると思うので。

import fs as "node:fs"import fs as "node:fs/promises"が衝突することもあると思います。
基本promisesな方をfsとし、前者はfsSyncとかにするのが良いかなと。
片方だけしかないならfsでも良いと思います。

その他

ここで少し話題に上がりました。

実装者募集中です!

@Hiroshiba Hiroshiba added 機能向上 初心者歓迎タスク 初心者にも優しい簡単めなタスク labels Mar 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
初心者歓迎タスク 初心者にも優しい簡単めなタスク 機能向上
Projects
None yet
Development

No branches or pull requests

1 participant