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.

Sign up for GitHub

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

Jump to bottom

一点点修改 #376

Merged
merged 12 commits into from
Jun 15, 2024
Merged

一点点修改 #376

merged 12 commits into from
Jun 15, 2024

Conversation

Fish-LP
Copy link
Contributor

@Fish-LP Fish-LP commented Jun 13, 2024

我朋友竟然找不到java11(

@allMagicNB
Copy link
Contributor

allMagicNB commented Jun 13, 2024 •
edited
Loading

时效性有点强，而且杂牌 JDK
说不准会有些啥问题（
而且 PCL 能自动安装 Java 啊，你这样反而是在让用户操作更加复杂……

@Fish-LP
Copy link
Contributor Author

Fish-LP commented Jun 13, 2024

是这里的https://adoptium.net/zh-CN/
pcl不能自动下载旧版java吧(应该?)

@allMagicNB
Copy link
Contributor

是这里的https://adoptium.net/zh-CN/
pcl不能自动下载旧版java吧(应该?)

你要不要先试试

LTCatt
LTCatt requested changes Jun 13, 2024
View reviewed changes
Copy link
Owner

@LTCatt LTCatt left a comment

Choose a reason for hiding this comment

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

“一点点修改”的那一部分没问题，可以留着。
添加 Java 下载……这个真的没必要。现在有自带的 Java 补全，可以应对大多数情况了。帮助库的核心目的还是为了服务小白玩家……这个还是删了吧 Orz。

@Fish-LP
Copy link
Contributor Author

Fish-LP commented Jun 14, 2024

留个java11吧，有的包需要，但是小白不会(比如我朋友)

@LTCatt
Copy link
Owner

LTCatt commented Jun 14, 2024

“我有一个朋友需要” 也不能说明这是 ”小白普遍存在的，以至于需要加入帮助库“ 的问题 :/

@Fish-LP
Copy link
Contributor Author

Fish-LP commented Jun 14, 2024

改好了我该怎么做 :|

allMagicNB
allMagicNB suggested changes Jun 14, 2024
View reviewed changes
帮助/提交帮助 - 编写规范.xaml Outdated Show resolved Hide resolved
帮助/提交帮助 - VSCode.xaml
Text="完成上述后，将第二步 clone 下仓库内 PCL2Help 文件夹中的内容剪切到 PCL\Help 中。在资源管理器中，在查看 → 显示下方选中 "隐藏的项目" 并将该文件夹内所有文件剪切至 PCL\Help 文件夹。"/>
<local:MyHint Margin="0,0,0,4" Text="在剪切后 .git 文件夹必须位于 PCL\Help，而不能在 PCL\Help\PCL2Help 中" />
Text="完成上述后，将第二步 clone 下仓库内 PCL2Help 文件夹剪切到 PCL 目录下并重命名文件夹 PCL2Help 为 Help 。"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

这段感觉还得改改，别直接删……

Copy link
Contributor Author

Choose a reason for hiding this comment

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

改了 : )

Fish-LP and others added 2 commits June 14, 2024 19:25
@Fish-LP
一点点修改
650a463
@Fish-LP @allMagicNB
应用代码审查中提出的建议
e8b9f69 
Co-authored-by: allMagic <116283787+allMagicNB@users.noreply.github.com>
@Fish-LP Fish-LP changed the title 添加 java 下载以及一点点修改 一点点修改 Jun 14, 2024
LTCatt
LTCatt requested changes Jun 14, 2024
View reviewed changes
帮助/提交帮助 - VSCode.xaml Outdated
@@ -42,8 +42,7 @@
Text="打开文件夹" EventType="打开文件" EventData="explorer.exe|/select,Help" />
</StackPanel>
<TextBlock Margin="0,0,0,15" LineHeight="17"
Text="完成上述后，将第二步 clone 下仓库内 PCL2Help 文件夹中的内容剪切到 PCL\Help 中。在资源管理器中，在查看 → 显示下方选中 &quot;隐藏的项目&quot; 并将该文件夹内所有文件剪切至 PCL\Help 文件夹。"/>
<local:MyHint Margin="0,0,0,4" Text="在剪切后 .git 文件夹必须位于 PCL\Help，而不能在 PCL\Help\PCL2Help 中" />
Text="完成上述后，将第二步 clone 下仓库内 PCL2Help 文件夹剪切到 PCL 目录下并重命名文件夹 PCL2Help 为 Help 。此外你可以选择在资源管理器中，在查看 → 显示下方选中 ”隐藏的项目“ 查看 .git 文件夹是否存在于 Help 文件夹"/>
Copy link
Owner

Choose a reason for hiding this comment

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

双引号反了，结尾没有句号

Copy link
Owner

Choose a reason for hiding this comment

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

此外你可以选择在资源管理器中，在查看 → 显示下方选中 ”隐藏的项目“ 查看 .git 文件夹是否存在于 Help 文件夹
这句话太绕口了……

Copy link
Contributor Author

Choose a reason for hiding this comment

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

改了(怎么引用代码段的啊)

LTCatt
LTCatt requested changes Jun 14, 2024
View reviewed changes
Copy link
Owner

@LTCatt LTCatt left a comment

Choose a reason for hiding this comment

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

此外，你可以选择在资源管理器中打开 选项卡查看 → 显示/隐藏 中的 “隐藏的项目”，以查看 .git 文件夹存在于 Help 文件夹内

帮助/提交帮助 - VSCode.xaml Outdated
@@ -42,7 +42,7 @@
Text="打开文件夹" EventType="打开文件" EventData="explorer.exe|/select,Help" />
</StackPanel>
<TextBlock Margin="0,0,0,15" LineHeight="17"
Text="完成上述后，将第二步 clone 下仓库内 PCL2Help 文件夹剪切到 PCL 目录下并重命名文件夹 PCL2Help 为 Help 。此外你可以选择在资源管理器中，在查看 → 显示下方选中 ”隐藏的项目“ 查看 .git 文件夹是否存在于 Help 文件夹"/>
Text="完成上述后，将第二步 clone 下仓库内 PCL2Help 文件夹剪切到 PCL 目录下并重命名文件夹 PCL2Help 为 Help 。此外你可以选择在资源管理器中打开 选项卡查看 → 显示/隐藏 中的 “隐藏的项目”选项 以查看 .git 文件夹是否存在于 Help 文件夹内"/>
Copy link
Owner

Choose a reason for hiding this comment

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

之后，将第二步中 clone 的仓库内 PCL2Help 的文件夹剪切到 PCL 目录下，并将 PCL2Help 文件夹重命名为 Help。
这一段感觉只需要改成这句话就行了，后面那句查看 .git 的都感觉是多余的？

Copy link
Contributor Author

Choose a reason for hiding this comment

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

我不知道.git是干嘛的()不确定重要性

Copy link
Contributor

Choose a reason for hiding this comment

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

配置 git 的……

Copy link
Contributor Author

Choose a reason for hiding this comment

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

一般不用管吗

Copy link
Contributor Author

Choose a reason for hiding this comment

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

.开头的文件默认隐藏 小白会觉得缺少文件吗

Copy link
Owner

Choose a reason for hiding this comment

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

这部分不需要用户操作，只要正确的剪切、重命名，这里就不会出问题，所以这句话都可以删了……
我个人是觉得把这一段直接换成 之后，将第二步中 clone 的仓库内 PCL2Help 的文件夹剪切到 PCL 目录下，并将 PCL2Help 文件夹重命名为 Help。 就行

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok，主要是我也不了解。

@LTCatt LTCatt merged commit 659e1d1 into LTCatt:master Jun 15, 2024
@Fish-LP
Copy link
Contributor Author

Fish-LP commented Jun 16, 2024

6D64-BB4A-F5A4-AB8A

@LTCatt
Copy link
Owner

LTCatt commented Jun 16, 2024

你的解锁码：Z+r1XUPeoXqFAPN5YjG9oSrnZelFWIVR
在 更多 → 关于与鸣谢 → 输入解锁码 中使用，谢谢支持～

@Fish-LP
Copy link
Contributor Author

Fish-LP commented Nov 7, 2024

853B-F3C1-43C6-D933

@LTCatt
Copy link
Owner

LTCatt commented Nov 7, 2024

你的解锁码：Ljog3QeVZjXDcDfUuH7WcQ0nY/6O3K6q
在 更多 → 关于与鸣谢 → 输入解锁码 中使用，谢谢支持～

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@allMagicNB allMagicNB allMagicNB requested changes

@LTCatt LTCatt LTCatt approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Fish-LP @allMagicNB @LTCatt