syunto07ka / jewelry_box

2 stars 0 forks source link

[wip]babel-plugin-react-css-modulesの導入 #48

Closed syunto07ka closed 4 years ago

syunto07ka commented 4 years ago

rel #20

注意: こちらclose予定

syunto07ka commented 4 years ago

@piro0919 一応close予定としています。 設定でこうしたほうがいいよみたいな部分あればご指摘をmm

piro0919 commented 4 years ago

あと、なぜかコミットログが全部日本語になっているのも良くないです ちょっと気を抜きすぎですね 🙅‍♂️

syunto07ka commented 4 years ago

あと、なぜかコミットログが全部日本語になっているのも良くないです ちょっと気を抜きすぎですね 🙅‍♂️

🙇

syunto07ka commented 4 years ago

すみません見落としでした🙇

(change request の中で指摘されると指摘事項のまとめ的なコメントだと思って読み飛ばしてしまうこと多いので、個人的には「ご確認ください」のようなお返しコメントに統一してほしい気持ち)

syunto07ka commented 4 years ago

語彙力

syunto07ka commented 4 years ago

https://github.com/syunto07ka/jewelry_box/pull/48#pullrequestreview-393208001 は e0d62be にて修正しました

piro0919 commented 4 years ago

@syunto07ka

(change request の中で指摘されると指摘事項のまとめ的なコメントだと思って読み飛ばしてしまうこと多いので、個人的には「ご確認ください」のようなお返しコメントに統一してほしい気持ち)

やです 😄 申し訳ないですが、見落とさないようにしてくださいなー 😸

syunto07ka commented 4 years ago

やです 😄

ww

piro0919 commented 4 years ago

@syunto07ka

別にいじわるで言っているわけではなくて レビューをあんまり堅苦しくするのがイヤなだけなので、あしからずです 😈

スコープを揃えて指摘するようにするので、一旦それで様子を見てもらっていいですか?

piro0919 commented 4 years ago

@syunto07ka

#48 (review)e0d62be にて修正しました

google的にはunderscoreよりhyphenを推奨しているので、しっかり覚えておいてください 👍 https://google.github.io/styleguide/htmlcssguide.html#ID_and_Class_Name_Delimiters

syunto07ka commented 4 years ago

スコープを揃えて指摘するようにするので、一旦それで様子を見てもらっていいですか?

すみませんお気を使わせて🙇 (最終的に合わせてくれるやさしさ)

syunto07ka commented 4 years ago

一回閉じて新しくまた上げ直したやつを見てもらいますmm