Skip to content

🔥BREAKING CHANGES おパッケージインポート時に乱数の初期化をしないように変更いたしましたわ🔥 #71#79

Merged
jiro4989 merged 6 commits into
mainfrom
feature/#71/non-init-rand
Aug 14, 2022
Merged

🔥BREAKING CHANGES おパッケージインポート時に乱数の初期化をしないように変更いたしましたわ🔥 #71#79
jiro4989 merged 6 commits into
mainfrom
feature/#71/non-init-rand

Conversation

@jiro4989

@jiro4989 jiro4989 commented Aug 14, 2022

Copy link
Copy Markdown
Owner

✨ 変更内容

  • パッケージインポート時に乱数の初期化をしないように変更
  • 乱数の初期化が必要な旨をREADMEに追記
  • Exampleテストを追加
  • cmd/ojosama に乱数の初期化処理を追加

close #71


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

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

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

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

💯 変更内容の適正性

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

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

♻️ 自動テストのパス

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

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

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

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

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

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

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

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


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

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

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

@codecov

codecov Bot commented Aug 14, 2022

Copy link
Copy Markdown

Codecov Report

Merging #79 (bec1c3a) into main (ae72600) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #79      +/-   ##
==========================================
- Coverage   96.86%   96.84%   -0.02%     
==========================================
  Files           5        5              
  Lines         478      476       -2     
==========================================
- Hits          463      461       -2     
  Misses         11       11              
  Partials        4        4              
Impacted Files Coverage Δ
ojosama.go 96.59% <ø> (-0.03%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@jiro4989 jiro4989 merged commit 141355a into main Aug 14, 2022
@jiro4989 jiro4989 deleted the feature/#71/non-init-rand branch August 14, 2022 08:51
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