Closed i-kaneko-abel closed 8 months ago
さまざまな修正が1コミットにまとまってしまっているので、何のために必要な修正かわかりづらいですね。 これだとPRをレビューする側も判断できないので、コミットを分けてそれぞれ何のために必要な修正なのかを書きましょう。
そもそも色々問題があるように見えますが、この内容であれば5コミットぐらいでしょうか。 (ものによってはPR自体を分けた方が良いかもしれませんが)
・型定義関係 ・外部からパッケージとしてimportするための設定(package.jsonのmain、index.ts) ・useEffectによるアイコン設定 ・defaultLatitudeなどの対応 ・allowImportingTsExtensionsの追加
package.json
main
index.ts
useEffect
defaultLatitude
allowImportingTsExtensions
また、過去のコミットを見た感じ、コミットコメントは英語の方が良さそうです。
PRを出した後はブランチ名の変更ができないですが、今回のブランチ名は内容と合っていないですね。 今回は仕方が無いとして、次からは作業内容に合わせた名前を考えましょうか。
適切なブランチ名に変更するため本PRはCloseといたします。
さまざまな修正が1コミットにまとまってしまっているので、何のために必要な修正かわかりづらいですね。 これだとPRをレビューする側も判断できないので、コミットを分けてそれぞれ何のために必要な修正なのかを書きましょう。
そもそも色々問題があるように見えますが、この内容であれば5コミットぐらいでしょうか。 (ものによってはPR自体を分けた方が良いかもしれませんが)
・型定義関係 ・外部からパッケージとしてimportするための設定(
package.json
のmain
、index.ts
) ・useEffect
によるアイコン設定 ・defaultLatitude
などの対応 ・allowImportingTsExtensions
の追加また、過去のコミットを見た感じ、コミットコメントは英語の方が良さそうです。