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

$status の表示を多少短くする #185

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

$status の表示を多少短くする #185

wants to merge 1 commit into from

Conversation

altnight
Copy link
Contributor

チケットURL

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

  • 表示の調整のみ

レビューチェックリスト

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

@altnight altnight self-assigned this Apr 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@wanshot wanshot Awaiting requested review from wanshot

Assignees

@altnight altnight

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@altnight