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

✨形容詞文に対応いたしましたわ❗ #12 #68

Merged
merged 2 commits into from
Jul 10, 2022

Conversation

jiro4989
Copy link
Owner

@jiro4989 jiro4989 commented Jul 10, 2022

✨ 変更内容

  • 専用の関数で形容詞文に対応していたところを、汎用の変換ルールで実装するように変更いたしましたわ

#12


⚠️ 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 #68 (0fe1c77) into main (1492318) will decrease coverage by 0.09%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #68      +/-   ##
==========================================
- Coverage   96.95%   96.86%   -0.10%     
==========================================
  Files           5        5              
  Lines         493      478      -15     
==========================================
- Hits          478      463      -15     
  Misses         11       11              
  Partials        4        4              
Impacted Files Coverage Δ
internal/converter/convert_rule.go 100.00% <ø> (ø)
ojosama.go 96.62% <100.00%> (-0.17%) ⬇️

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 1492318...0fe1c77. Read the comment docs.

@jiro4989 jiro4989 merged commit a77582e into main Jul 10, 2022
@jiro4989 jiro4989 deleted the feature/#12/keiyoshi branch July 10, 2022 02:52
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