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

New feature: SkillTooltipComparison #205

Open
HikariCalyx opened this issue Jul 7, 2022 · 2 comments
Open

New feature: SkillTooltipComparison #205

HikariCalyx opened this issue Jul 7, 2022 · 2 comments

Comments

@HikariCalyx
Copy link
Contributor

这是基于 https://github.com/seotbeo/WzComparerR2/tree/skill 分支移植的功能。
目前这个更改我已经在 PirateIzzy 的英文版分支下移植完成了,但修改的程度有点大。
以下是相关Pull request。

https://github.com/PirateIzzy/WzComparerR2/pull/9

@Kagamia
Copy link
Owner

Kagamia commented Jul 7, 2022

很想开一个review。。但在别人的repo中review不太礼貌,于是手动描述一下可见的问题:

  1. vs2022可以安装各种版本的.net framework devpack(除了已声明过时的4.62前的版本),无需强制升级到4.8。
  2. 所有的csproj文件的diff都没有被正确对比,这可能和git的autocrlf相关。
  3. EasyComparer.cs文件无法正确对比,甚至无法正常review到底改了什么。
  4. FrmPatcher.Designer.cs文件的改动中,出现了非本土化UI提示内容。
  5. CharaSimLoader.cs文件的改动中,存在未正确命名的变量Wz_Node wzNode,并且没有为前向兼容添加默认值或重载。
  6. StringLinker.cs文件的改动中,出现了无法理解的新增参数,但却没有使用过。同理所有的关联改动。
  7. Readme.md文件的改动中标注自己为contributor,似乎不太符合礼仪(个人感受)。

@HikariCalyx
Copy link
Contributor Author

了解,先放着看看,目前只是先私下做了一个修改build发现可行。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants