Skip to content

Commit

Permalink
Update CONTRIBUTING.md
Browse files Browse the repository at this point in the history
  • Loading branch information
syuilo committed Aug 23, 2021
1 parent 4218a5b commit e5f7bda
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ PRを作る際は、`npm run api`コマンドを実行してAPIレポートを

## レビュイーの心得
PRを作成するときのテンプレートに色々書いてあるので読んでみてください。(このドキュメントに移してもいいかも?)
また、後述の「レビュー観点」も意識してみてください。

## レビュワーの心得
- 直して欲しい点だけでなく、良い点も積極的にコメントしましょう。
Expand All @@ -26,4 +27,7 @@ PRを作成するときのテンプレートに色々書いてあるので読ん
- パフォーマンス
- このPRをマージすることで、予期せずパフォーマンスが悪化しないか?
- もっと効率的な方法は無いか?

- テスト
- 期待する振る舞いがテストで担保されているか?
- 抜けやモレは無いか?
- 異常系のチェックは出来ているか?

0 comments on commit e5f7bda

Please sign in to comment.