syunto07ka / jewelry_box

2 stars 0 forks source link

package.jsonのstylelint設定項目のスタイルの修正 #72

Closed syunto07ka closed 4 years ago

syunto07ka commented 4 years ago

スペースなかったので足す

syunto07ka commented 4 years ago

@piro0919 瞬殺のやつですmm

syunto07ka commented 4 years ago

さっきmasterから切ったブランチでnpm install 叩くとspace差分出ること確認しました

 ~/development/jewelry_box  set_cord_formater  0m  npm install --save prettier                                                                                                                       
+ prettier@2.0.5
added 1 package from 1 contributor and audited 2780 packages in 21.251s

129 packages are looking for funding
  run `npm fund` for details

found 5 low severity vulnerabilities
  run `npm audit fix` to fix them, or `npm audit` for details
 ~/development/jewelry_box  set_cord_formater●  0m 

 ~/development/jewelry_box  set_cord_formater●  0m  git diff package.json                                                                                                                            
diff --git a/package.json b/package.json
index 5ec0f65..90212cb 100644
--- a/package.json
+++ b/package.json
@@ -56,6 +56,7 @@
     "postcss-preset-env": "6.7.0",
     "postcss-safe-parser": "4.0.1",
     "postcss-scss": "2.0.0",
+    "prettier": "2.0.5",
     "react": "^16.13.1",
     "react-app-polyfill": "^1.0.6",
     "react-dev-utils": "^10.2.1",
@@ -170,7 +171,7 @@
     ]
   },
   "stylelint": {
-    "extends":[
+    "extends": [
       "stylelint-config-standard",
       "stylelint-config-sass-guidelines"
     ],
 ~/development/jewelry_box  set_cord_formater●  0m 

へぇnpm installしたらpackage.jsonのコード整形やってくれるんだという学び

syunto07ka commented 4 years ago

@piro0919 jsonもコミット前にコードフォーマットがかかるようにする。とかですか?

piro0919 commented 4 years ago

@syunto07ka

jsonもコミット前にコードフォーマットがかかるようにする。とかですか?

ちょっと考え方が正しくないですね 正しくはlinterあたりで縛ったりするのが良いと思います(json向けのlinterを知らないですが…

それはそれとして賛否両論あるとは思いますが、個人的にはやはりvscodeのprettierが最強かなぁと思います 今回は手動で直されたと思いますが、それは一番やってはいけない修正方法だということを念頭に置いておきましょう 👍

syunto07ka commented 4 years ago

今回は手動で直されたと思いますが、それは一番やってはいけない修正方法だということを念頭に置いておきましょう 👍

なるほどです 👀