改）README書き換え。作成者の意図やルール - Githubissues

shari-sushi / V-Kara-Lists

Vカラ（Vtuber歌枠把握アプリ）

MIT License

16 stars 4 forks source link

改）README書き換え。作成者の意図やルール #176

Open shari-sushi opened 5 months ago

shari-sushi commented 5 months ago

※既存branch README&figureでやること

現状、READMEの内容はQiitaと丸かぶりしているうえ、GitHubのREADMEっぽくない。内容変更を検討している。

候補
- publicにしている理由
- 初学者向け
- 注意点…vtuber-karaの書き方の間違いとか
- ローカルサーバーの立ち上げ方
- ライセンスの要約？や選定理由
- 商用利用等の希望
- 展望？
- Contributingに対するルール：参考discordGo
- issueのlabelのルール

shari-sushi commented 3 months ago

コーティング規約

upate, createの時のrespons ok の際はその結果のデータも返す。

プルリク前に見直す

(ドラフトプルリクエストにして、レビューしないで良い旨を示すのでもok)

yarn lint
yarn tsc --Emit
react初心者向け
- hooksは条件分岐前で宣言すること
- throw Errorはhooksの手前でも良さそう。条件分岐後にhooksを宣言しちゃいけないのは、前回/次回のレンダリングでhooksを呼ばれた順番に比較するのがうまくいかなくなるから(参照)。throw投げたら比較しなくて良くなるので大丈夫そ？
- メモ化してるか(コンポーネントに渡す関数は全てメモ化する)
- hooks系の依存配列の過不足
- useEffect本当に必要か。→callBackとuseStateでなんとかなる。
いろいろ初心者向け
- hoge関数内でhogeHugaという変数名を付けてないか(hugaで良い)
- 文字列の誤字
- 不要なexport

merge

以下のどちらでも良い

プルリクタイトルで内容を示してsquash merge
コミットメッセージをちゃんと書いて普通にmerge

shari-sushi commented 3 months ago

一生手が付かなさそうなので、「簡易でもいいからとりあえず」実装する。目安、30分～１時間

shari-sushi commented 3 months ago

コマンドメモ

mysql -uroot -ppassword --host 127.0.0.1