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

印板固有処理 #470

Merged
merged 16 commits into from
Dec 10, 2024
Merged

印板固有処理 #470

merged 16 commits into from
Dec 10, 2024

Conversation

NePonpon
Copy link
Member

@NePonpon NePonpon commented Nov 20, 2024

チケットURL

#425

対応内容・対応背景・妥協点

  • 印板の固有処理のアップデート対応をした

やったこと

  • functions -> function


  • 花火演出のマクロ化

テスト

  • 共通処理がこのプルリクに含まれていないため、テスト不可でした

レビュー観点

  • 正しく動作するように記述されている。

チェックリスト

  • PRの必須項目はすべて記載していますか?
    • 必要ない項目は削除していますか?
  • PRの内容と変更内容は一致していますか?
    • 1機能=1PRであるべきです。1機能の途中でも問題ありません
  • コミットの粒度は適切ですか?
    • 1コミット=1要素(1ロジック)の変更であるべきです
  • PRのタイトルはわかり易い名前が設定されていますか?(日本語でも可)
  • ブランチの向き、切り先は正しいですか?
  • ブランチの名前は正しいですか?
    • ブランチ名先頭はfeature/[簡単な説明]fix/[簡単な説明] の何れかであるべきです
    • 簡単な説明は英語であるべきです(不具合発生するので)
  • Commitメッセージはルール通りですか?
    • コミットメッセージの先頭に[Add|Delete|Modify|Fix|Refactor|Move]等の動詞の原形を追加してください。(参考)
    • コミットメッセージの説明にはGH-〇〇でチケット番号をつけてください(チケットが存在しない場合はNO-ISSUEにしてください
    • マージコミットの場合はこの限りではありません

@NePonpon NePonpon merged commit fd12b10 into 1-21-1-dev Dec 10, 2024
2 checks passed
@NePonpon NePonpon deleted the feature/#425/印板固有処理 branch December 10, 2024 13:42
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code review by ChatGPT

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