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

めり込み死亡のチュートリアルを追加 #704

Merged
merged 3 commits into from
Feb 16, 2025

Conversation

NePonpon
Copy link
Member

@NePonpon NePonpon commented Feb 13, 2025

チケット URL

GH-703

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

「おあっと!」のチュートリアル進捗を追加しました。
めり込み死亡時に達成されます。

やったこと

  • secret19としてチュートリアル進捗を追加
  • めり込み死亡処理に進捗達成コマンドを追加

テスト

めり込みはスペクテイターモードを使ってテストするほうがわかりやすいです。
エンダーパールでのテストは、めり込み抑制機能によってやりづらいです。

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

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

Copy link
Contributor

@yabane yabane left a comment

Choose a reason for hiding this comment

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

2025-02-15_22 27 07
問題なく動作はしたけど、バリアーブロック→バリアブロックで修正すると読みやすくなって表現も自然になります

@anminmakura
Copy link
Contributor

anminmakura commented Feb 15, 2025

あ~バリアブロックの方が見やすいかぁ...という訳でシートの方は直しました。

バリアーブロック→バリアブロックで修正すると読みやすくなって表現も自然になります

@NePonpon
出力後でお願いしたいですー

@NePonpon
Copy link
Member Author

せっかくならバリアブロックの翻訳を利用しましょう

@NePonpon
Copy link
Member Author

岩盤とバリアブロックのブロック名を翻訳キーに変更しました。

@NePonpon NePonpon requested a review from yabane February 15, 2025 14:24
@NePonpon NePonpon merged commit 6d79456 into update/v13-2-5-alpha2 Feb 16, 2025
3 checks passed
@NePonpon NePonpon deleted the feature/suffocation-tutorial branch February 16, 2025 04:19
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.

[feature] 進捗「おあっと!」を追加 (岩盤やバリアーブロックにめり込むと即死)
3 participants