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

モブへの攻撃時処理を追加 #677

Draft
wants to merge 1 commit into
base: 1-21-1-dev
Choose a base branch
from

Conversation

NePonpon
Copy link
Member

チケット URL

NO-ISSUE

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

プレイヤーがモブへ攻撃を与えた時の処理を実装しました。

やったこと

  • functions -> function

やってないこと

投擲物でのダメージについて、ダメージの受け渡しが他方の変更で不十分になっているので、完全に動作しません。
その他、ダメージに関係するトリガーを全部見直してからこの実装についてのデバッグを行います。

チェックリスト (ガイドライン)

  • Pull Request

    • PR のタイトルはわかりやすい名前が設定されていますか?(日本語でも可)
    • PR の必須項目はすべて記載していますか?
    • PR の必要ない項目は削除していますか?
    • PR の内容と変更内容は一致していますか?
      • 1 機能=1PR であるべきです。1 機能の途中でも問題ありません
  • Commit

    • Commit メッセージはルール通りですか?
      • コミットメッセージの先頭に[Add|Delete|Modify|Fix|Refactor|Move]
        等の動詞の原形を追加してください。
        (参考)
        (参考 2)
      • コミットメッセージの説明にはGH-〇〇でチケット番号をつけてください
        (チケットが存在しない場合はNO-ISSUEにしてください)
      • マージコミットの場合はこの限りではありません
    • コミットの粒度は適切ですか?
      • 1 コミット=1 要素(1 ロジック)の変更であるべきです
  • Branch

    • ブランチの名前は正しいですか?
      • ブランチ名先頭はfeature/[簡単な説明] または fix/[簡単な説明] であるべきです
      • 簡単な説明は英語であるべきです(不具合発生するので)
    • ブランチの向き、切り先は正しいですか?

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

Successfully merging this pull request may close these issues.

1 participant