-
Notifications
You must be signed in to change notification settings - Fork 305
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
build下のスクリプトをtoolsに動かしてTypeScriptにする #2361
Conversation
🚀 プレビュー用ページを作成しました 🚀 更新時点でのコミットハッシュ: |
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.
ほぼLGTMです!!
整理ありがとうございます!!
docsのコードの歩き方.md
のソースコードのディレクトリ構成
の説明にtools
のことを書いてもいいかもですね。
まあちょっとメンテされてなさすぎて他の情報が古くなりすぎてますが。。。 😇
ま、まあこれは気が向いたらくらいの気持ちで。。。
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.
LGTM!!
リファクタリングありがたい!!!!!
どんどんやっていきたいのでどんどん提案いただけると!!!!
mkdirSync(TYPOS_BINARY_PATH, { recursive: true }); | ||
await fs.mkdir(TYPOS_BINARY_PATH, { recursive: true }); |
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.
(ただのコメントです)
asyncな関数をawaitするより、syncな関数をそのまま使うほうが実は便利なのではとちょっと思ってたりします。
asyncが広まらないので。
あっすみません、コンフリクトの解消お願いしてもいいですか 🙇 |
マージしました。 |
内容
build下のスクリプトをtoolsに動かします。
関連 Issue
(なし)
スクリーンショット・動画など
(なし)
その他
(なし)