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

💚循環的複雑度のテストを追加いたしましたわ♻️ #65 #69

Merged
merged 2 commits into from
Jul 10, 2022

Conversation

jiro4989
Copy link
Owner

@jiro4989 jiro4989 commented Jul 10, 2022

✨ 変更内容

  • 循環的複雑度21以上はエラーにいたしますわ

⚠️ Pull Requestを作成する際のお約束事 ⚠️

Pull Request(以降PR)を作成する際のお約束事について書いておりますわ。
ちょっとだけお付き合いくださいましね。

🛡️ 二次創作おガイドラインの遵守

ojosama はにじさんじ所属の壱百満天原サロメお嬢様のキャラクターを題材にした二次創作の一つですわ。
故に、以下二次創作おガイドラインに従いますわ。

💯 変更内容の適正性

その変更が「壱百満天原サロメお嬢様の口調の再現に必要な変更かどうか」を確認いたしますわ。

例えば全然違う方の口調を再現するような変更の場合は、お却下する場合がございますの。
こちらは、このリポジトリのおポリシーになりますので、ご了承くださいましね。

♻️ 自動テストのパス

PRが作成されると必ず自動テストが実行されますの。
そして、可能な限り単体テストがパスする状態にしてくださいまし。

単体テストがパスしていない場合、私からもレビューいたしますが、
レビューに対して一週間以上返信をいただけないと、私の方でPRを却下するかもしれませんわ。

また、変換ルールを追加した場合は、なるべくテストコードも追加してくださいましね。

おテスト網羅率が下がってしまうのは仕方のない場合もありますので、あまり気にしておりませんわ。
ただし、テストコードの追加もれでおテスト網羅率が下がっている場合は、私も指摘いたしますのでご対応くださいましね。

❤️ 様々な方からのPRの歓迎

現時点(2022/06/27)で、たくさんのエンジニアお嬢様の皆様方からPRをいただいておりますが、
もちろん、お嬢様でない方からのPRもお受け取りいたしますわ。

お嬢様な言葉遣いが分からなくても、お気になさらずにバシバシ送りつけてくださいまし。

言葉遣いを理由に却下したり、変更を依頼することは決してございません❗


✅ レビューワー様用のおチェックリスト

レビューワー様は以下おチェックリストを確認してくださいまし。

  • 二次創作おガイドラインの遵守
  • 変更内容の適正性
  • 自動テストのパス
  • 様々な方からのPRの歓迎

@codecov
Copy link

codecov bot commented Jul 10, 2022

Codecov Report

Merging #69 (0b52314) into main (a77582e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #69   +/-   ##
=======================================
  Coverage   96.86%   96.86%           
=======================================
  Files           5        5           
  Lines         478      478           
=======================================
  Hits          463      463           
  Misses         11       11           
  Partials        4        4           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a77582e...0b52314. Read the comment docs.

@jiro4989 jiro4989 changed the title 💯循環的複雑度のテストを追加いたしましたわ♻️ #65 💚循環的複雑度のテストを追加いたしましたわ♻️ #65 Jul 10, 2022
@jiro4989 jiro4989 merged commit 0d038ca into main Jul 10, 2022
@jiro4989 jiro4989 deleted the chore/#65/gocyclo branch July 10, 2022 03:36
@jiro4989
Copy link
Owner Author

close #65

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

Successfully merging this pull request may close these issues.

1 participant