Skip to content
This repository has been archived by the owner on Aug 17, 2023. It is now read-only.

[WIP] MySQL 5.7 対応 #127

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

[WIP] MySQL 5.7 対応 #127

wants to merge 4 commits into from

Conversation

altnight
Copy link
Contributor

チケットURL

このレビューで確認してほしい点

  • foo bar
  • hoge fuga

レビューチェックリスト

  • [skip] C2 体を表す名前の公理:あらかじめ決められている以外の汎用的な名前のモジュールを作らない
  • [skip] C3 汎用名のモジュール内に長々と具体的処理を書かない
  • [skip] C4 単純な処理の長さで分割しない
  • [skip] C5 引数の数を減らす
  • [skip] C6 パッケージ間で共通した定数を作らない
  • [skip] C7 継承の利用を最小限にする
  • [skip] C8 親クラスのテストを子クラスでも実行すること
  • [skip] C9 オーバーライドを減らす
  • [skip] C10 継承やオーバーライドを明示する

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant