Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: detekt rule の更新 #122

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

Conversation

kinari321
Copy link
Contributor

@kinari321 kinari321 commented Jan 21, 2023

全体的な意図 ( What, How )

  • detekt のルールを一部緩める

追加・変更の文脈、位置づけ。なぜこの追加・変更が必要か ( Why )

  • コンストラクタの引数の数、関数の引数のデフォルトの数が少ないため

この PR がマージされると何が嬉しいか

  • akiyadego-kotlin の Post クラスのファクトリメソッドが作成できるようになる

Checklist

追加・変更したテストケースの見出し一覧

  • 特に無し

Issue

レビュアーに手動 QA 作業してほしいですか

  • QA 作業をして欲しい( MUST: QA 手順 )
    • 加えて、記述した手順も見て欲しい
  • QA 作業をしなくてもよい
    • ただ、記述した手順は見て欲しい

QA 作業手順

レビューする時

コメントする時

以下のようなラベルをつけると温度感や、ざっくりと伝えたいことががわかります(小文字でも OK です。厳密な使い分けは不要です)

  • MUST: 必ず修正・変更して欲しい
  • WANT: できれば修正・変更して欲しい
  • IMO: (In my opinion) 私の意見では
  • IMHO: (In my humble opinion) 私のつたない意見では
  • nits: (nitpick) ほんの小さな指摘。インデントミスなどの細かいところに。
  • ASK: 質問。わからないことがあれば質問してみましょう。
  • FYI: (For Your Informatio) 参考までに
  • GOTCHA: やったぜ
  • NP: 問題ない

コピペ用絵文字

https://emojipedia.org/

😀:にっこり
😖:困った
🤔:むむ

👍:(・∀・)イイネ!!

@kinari321 kinari321 self-assigned this Jan 21, 2023
@kinari321 kinari321 linked an issue Jan 21, 2023 that may be closed by this pull request
@kinari321 kinari321 marked this pull request as ready for review January 21, 2023 15:31
@kinari321 kinari321 requested a review from a team January 21, 2023 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

detekt の設定の更新
1 participant