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

固定商人《貴重品》バンクの更新 #718

Merged
merged 3 commits into from
Feb 22, 2025

Conversation

anminmakura
Copy link
Contributor

@anminmakura anminmakura commented Feb 21, 2025

チケット URL #717

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

タイトル通り
固定商人《貴重品》バンクに

  • プリズムソーテイラー
  • オールドイブリース
  • ブラッド・メナス
    一式のメモリスターを取引に加える。

やったこと

スプレッドシートの1.19.4及び1.21.x以降の《貴重品》バンクの取引内容の追加及び出力
1.21.xの出力はまだ。

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

  • 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/[簡単な説明] であるべきです
      • 簡単な説明は英語であるべきです(不具合発生するので)
    • ブランチの向き、切り先は正しいですか?

@anminmakura anminmakura added 1-19-4 MCver1.19.4で確認された問題 優先度: 通常 プリーズレビュー レビューほしいときに使う(PRの場合はOpenAI 4oによるレビューが始まります) リリース予定 labels Feb 21, 2025
@anminmakura anminmakura requested a review from a team as a code owner February 21, 2025 11:04
@anminmakura anminmakura enabled auto-merge (squash) February 21, 2025 11:04
Copy link
Member

@NePonpon NePonpon left a comment

Choose a reason for hiding this comment

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

メモリスターではなく、アメジストの欠片のままになっているようです。
シート上でもメモリスターになっていないようなので確認してみてください

@anminmakura
Copy link
Contributor Author

確かに売品がアメジストの欠片になっていました。
これを修正するコミットを出力します。

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

Copy link
Member

@NePonpon NePonpon left a comment

Choose a reason for hiding this comment

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

取引の追加を確認しました。

@anminmakura anminmakura merged commit 8fc2839 into update/v13-2-5-alpha2 Feb 22, 2025
3 checks passed
@anminmakura anminmakura deleted the fix/employee_bank_update branch February 22, 2025 03:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1-19-4 MCver1.19.4で確認された問題 プリーズレビュー レビューほしいときに使う(PRの場合はOpenAI 4oによるレビューが始まります) リリース予定 優先度: 通常
Projects
None yet
2 participants