修正副产物的计算逻辑 #13
Merged
修正副产物的计算逻辑 #13
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
|
顺道更正以后把0.4.5的tag换成这个吧 |
JimmyZJX
reviewed
Jan 9, 2024
package.json
Outdated
There was a problem hiding this comment.
啊,我这里是因为原先我本地的环境跑不了改了webp以后的那个版本,然后按它的报错在网上找教程然后一个个手动install才解决的,都没注意吧这里也改了,把这一步删了吧
|
我觉得还是不要改以前release过的版本号，还是+1再release一个新版本吧 |
主要现在的0.4.5计算逻辑上有问题,这不是布局之类的小问题,最好还是别有这个基础功能上出问题的版本,不过我也拿不准主要,这种情况还是按你想的来吧 |
|
我觉得正是因为有bug所以应该新开一个版本，这样就能区分出来用户给的bug report到底是什么原因引起的 |
行,那就这么办吧 |
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.
当初更改item_price的时候把副产物的收获一起算到item_price里了,然后我记得下面一段计算副产物收获的内容删了或注释了,后来不知道可能是哪一次合并又把这段加了回来,导致副产物产出算了两次,现在这边再次删掉