Skip to content

Commit

Permalink
📝 テンプレートの説明が冗長だったのでコンパクトにいたしましたわ~❗
Browse files Browse the repository at this point in the history
  • Loading branch information
jiro4989 committed Jun 27, 2022
1 parent 2bd04a0 commit ca31189
Showing 1 changed file with 2 additions and 17 deletions.
19 changes: 2 additions & 17 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,13 +16,6 @@ Pull Request(以降PR)を作成する際のお約束事について書いてお

* [ANYCOLOR二次創作ガイドライン](https://event.nijisanji.app/guidelines/)

ですので、PRのおレビューをする際も、上記おガイドラインを守っているかどうかを
まず最初に確認いたしますわ。

特に変換結果が誰かに迷惑をかけたり、
壱百満天原サロメお嬢様のイメージを損なうようなものの場合は、
お却下する場合がございますわ。

### :100: 変更内容の適正性

その変更が「壱百満天原サロメお嬢様の口調の再現に必要な変更かどうか」を確認いたしますわ。
Expand All @@ -33,22 +26,14 @@ Pull Request(以降PR)を作成する際のお約束事について書いてお
### :recycle: 自動テストのパス

PRが作成されると必ず自動テストが実行されますの。
自動テストには以下の内容が含まれますわ。

- 単体テスト
- お静的解析
- コードフォマット済みかどうか
- おビルドが通るかどうか
- `go test` が通るかどうか
- おテスト網羅率の算出

**単体テストがパスしない状態では決してPRをおマージいたしませんわ。**
これは絶対にでございます。

単体テストがパスしていない場合、私からも指摘いたしますが、
一週間以上返答がないのであれば、一方的にこちらからPRを却下いたしますわ。

特に、変換ルール周りの変更の場合は自動テストの結果を注意深く確認いたしますわ。
変換ルールを追加した場合は、必ずテストコードも追加してくださいましね。
特に、変換ルールを追加した場合は、必ずテストコードも追加してくださいましね。

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

0 comments on commit ca31189

Please sign in to comment.