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

💯リファクタリングですわ! #64

Merged
merged 8 commits into from
Jul 7, 2022
Merged

💯リファクタリングですわ! #64

merged 8 commits into from
Jul 7, 2022

Conversation

jiro4989
Copy link
Owner

@jiro4989 jiro4989 commented Jul 7, 2022

✨ 変更内容

  • convert_rule, convert_condition が肥大化してきたのでパッケージを分けた
    • これらのルールを生成するためだけの関数がそれなりに増えてきて、且つそれらは ojosama.go から見られるべきではない

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

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

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

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

💯 変更内容の適正性

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

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

♻️ 自動テストのパス

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

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

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

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

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

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

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

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


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

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

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

@codecov
Copy link

codecov bot commented Jul 7, 2022

Codecov Report

Merging #64 (e9db7ad) into main (afe947b) will increase coverage by 1.82%.
The diff coverage is 100.00%.

❗ Current head e9db7ad differs from pull request most recent head 71e2fa3. Consider uploading reports for the commit 71e2fa3 to get more accurate results

@@            Coverage Diff             @@
##             main      #64      +/-   ##
==========================================
+ Coverage   95.13%   96.95%   +1.82%     
==========================================
  Files           5        5              
  Lines         493      493              
==========================================
+ Hits          469      478       +9     
+ Misses         17       11       -6     
+ Partials        7        4       -3     
Impacted Files Coverage Δ
internal/converter/convert_condition.go 100.00% <100.00%> (ø)
internal/converter/convert_rule.go 100.00% <100.00%> (ø)
internal/tokendata/util.go 100.00% <100.00%> (ø)
ojosama.go 96.78% <100.00%> (-0.11%) ⬇️

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 afe947b...71e2fa3. Read the comment docs.

@jiro4989 jiro4989 merged commit 0647c36 into main Jul 7, 2022
@jiro4989 jiro4989 deleted the chore/refactor branch July 7, 2022 17:09
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