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

#215 blackを導入 #217

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

#215 blackを導入 #217

wants to merge 2 commits into from

Conversation

kemu3007
Copy link

@kemu3007 kemu3007 commented Nov 2, 2020

チケットURL

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

  • blackを用いたformat, ciでのテストが走るようになっている

レビューチェックリスト

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

@kemu3007 kemu3007 self-assigned this Nov 2, 2020
@@ -26,25 +26,36 @@ commands = pytest {posargs}
commands = pytest {posargs} \
--junitxml={toxinidir}/test-results/pytest.xml

[testenv:flake8]
[testenv:lint]
deps =
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lint, fmt, lint-ciコマンドを追加しました、これまで利用していたflake8, flake8_ciはlint, lint_ciに統合されています

@kemu3007 kemu3007 requested review from a team and jamesvandyne and removed request for a team November 2, 2020 07:38
Copy link
Contributor

@jamesvandyne jamesvandyne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

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

Successfully merging this pull request may close these issues.

2 participants