-
-
Notifications
You must be signed in to change notification settings - Fork 2
Call本体機能の追加 #794
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
base: releases/1-21-1
Are you sure you want to change the base?
Call本体機能の追加 #794
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR implements the call functionality by migrating Call‐system components to the AI space and updating corresponding identifiers.
- Updated GitHub workflow YAML configuration for improved multi‐line formatting.
- Renamed skill identifiers from legacy “skill:enemy/” prefixes to the new “ai:skill/” pattern.
- Minor formatting adjustments in the YAML file to enhance clarity and consistency.
Files not reviewed (19)
- data/ai/function/call/call.mcfunction: Language not supported
- data/ai/function/call/condition/condition/.mcfunction: Language not supported
- data/ai/function/call/condition/condition/check_block.mcfunction: Language not supported
- data/ai/function/call/condition/condition/check_nbt.mcfunction: Language not supported
- data/ai/function/call/condition/condition/function.mcfunction: Language not supported
- data/ai/function/call/condition/condition/hp.mcfunction: Language not supported
- data/ai/function/call/condition/condition/min_hp.mcfunction: Language not supported
- data/ai/function/call/condition/condition/min_lv.mcfunction: Language not supported
- data/ai/function/call/condition/condition/min_mp.mcfunction: Language not supported
- data/ai/function/call/condition/condition/mp.mcfunction: Language not supported
- data/ai/function/call/condition/condition/random_chance.mcfunction: Language not supported
- data/ai/function/call/condition/condition/target/.mcfunction: Language not supported
- data/ai/function/call/condition/condition/target/half.mcfunction: Language not supported
- data/ai/function/call/condition/logic/.mcfunction: Language not supported
- data/ai/function/call/condition/logic/and.mcfunction: Language not supported
- data/ai/function/call/condition/logic/or.mcfunction: Language not supported
- data/ai/function/call/condition/logic/single.mcfunction: Language not supported
- data/ai/function/call/execute/.mcfunction: Language not supported
- data/ai/function/call/execute/all.mcfunction: Language not supported
Comments suppressed due to low confidence (1)
.github/workflows/chatgpt-review.yml:136
- [nitpick] The renaming to the 'ai:skill' prefix enhances consistency; please verify that this update is reflected uniformly across relevant documentation and references.
- - `ai:skill/laser/loop/security_laser`
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
チケット URL
GH-790
対応内容・対応背景・妥協点
skill:enemy/
にあったスキル実装をai空間へ移動する。やったこと
functions
->function
enemy:ai/call/
->ai:/call/
skill:enemy/
->ai:skill/
やってないこと
テスト
Function
を使っていてその中身が更新されていない場合の動作は保証されませんレビュー観点
チェックリスト (ガイドライン)