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

Delete 无法更新 #59

Closed
wants to merge 1 commit into from

Conversation

allMagicNB
Copy link
Contributor

这玩意没用,PCL2 的更新压根就不是从 GitHub 下载的,而且一般来说不会有第二个人遇到这个问题,遇到了也不会看这玩意,这玩意还具有误导性

@allMagicNB allMagicNB requested a review from WForst-Breeze June 15, 2024 05:13
@WForst-Breeze WForst-Breeze added the ★ 表格内容 补充、更新、调整、完善表格正文内容 label Jun 15, 2024
@WForst-Breeze
Copy link
Collaborator

这两天忙没时间,有时间再来看

@wuliaodexiaoluo
Copy link
Member

根据 #3505 日志内出现的 GitHub 链接的网络请求结果,我认为正式版更新使用的是 GitHub Raw 和 GitCode Raw 作为更新源

[16:36:04.570] [Download] Update.zip 81#:开始,起始点 0,https://github.com/Hex-Dragon/PCL2/raw/main/%E6%9C%80%E6%96%B0%E6%AD%A3%E5%BC%8F%E7%89%88.zip

因此我认为此条目没有删除的需要,但确实应当予以重写

@wuliaodexiaoluo wuliaodexiaoluo added the ◈ 争议 该内容尚无法确定其必要性 & 存在争议,须经考虑后再议 label Jun 15, 2024
@WForst-Breeze WForst-Breeze added the ⇲ 关联 该内容与其它元素有一定关联性 label Jun 15, 2024
Copy link
Collaborator

@WForst-Breeze WForst-Breeze left a comment

Choose a reason for hiding this comment

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

参阅 Hex-Dragon/PCL2#4043,采取小洛的意见,该条目不进行删除,但是的确需要重写(正文+参考)并对其进行特别的标注以避免误导性。

先关掉了,感谢!

@WForst-Breeze WForst-Breeze added × 拒绝 该内容因为一些原因无必要修改、添加或与现有 Pull Request 重复 × 重新编写 此内容存在进行修改的必要性,但该 Pull Request 给予的解决方案欠妥,需要维护者或社区后续重新进行编辑 and removed ◈ 争议 该内容尚无法确定其必要性 & 存在争议,须经考虑后再议 × 拒绝 该内容因为一些原因无必要修改、添加或与现有 Pull Request 重复 labels Jun 20, 2024
@allMagicNB allMagicNB deleted the allMagicNB-patch-2 branch July 11, 2024 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
× 重新编写 此内容存在进行修改的必要性,但该 Pull Request 给予的解决方案欠妥,需要维护者或社区后续重新进行编辑 ⇲ 关联 该内容与其它元素有一定关联性 ★ 表格内容 补充、更新、调整、完善表格正文内容
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants