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

#214 URLを含む語録が削除できないバグ修正 #216

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kemu3007
Copy link

@kemu3007 kemu3007 commented Nov 2, 2020

チケットURL

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

  • データマイグレーションが適切に書かれている

レビューチェックリスト

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

動作確認
image

@kemu3007 kemu3007 self-assigned this Nov 2, 2020
@kemu3007 kemu3007 added the bug label Nov 2, 2020
@kemu3007 kemu3007 requested a review from wanshot November 2, 2020 05:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant