cost事件新增skill_popup属性；技能修改 #1439
Merged
+67
−51
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR受影响的平台
所有平台
诱因和背景
①手杀公告の【秉清】结算描述修改
②cost事件对于是否logSkill的自由度不高
③日本知名漫画家jjxx开始接手OL武将后续调整[doge]
PR描述
①修改毛玠【秉清】结算
②cost事件新增
skill_popup属性，当event.result存在skill_popup属性且Boolean(skill_popup)为false时，将不执行logSkill操作③添加OL为裴秀添加的补丁（OL你无敌了）
PR测试
跑了能过
扩展适配
无
检查清单
character/rank.js中添加对应的武将强度评级/在lib.translate中加入语音文件的文字台词PR描述中写入详细文档扩展适配中写入详细文档诱因和背景中明确链接到该issue.editorconfig、eslint.config.mjs和prettier.config.mjs所规定的代码样式，并且已经通过prettier格式化过代码