Skip to content

Conversation

@TyMxy
Copy link
Contributor

@TyMxy TyMxy commented Nov 24, 2025

对文章注意事项中最低版本号进行了修正,以符合现状。

对文章注意事项中最低版本号进行了修正,以符合现状。
@TyMxy
Copy link
Contributor Author

TyMxy commented Nov 24, 2025

另外我想问一下,对于文档中最低版本的声明,应该写最低正式版还是最低测试版还是两个都写?

@3gf8jv4dv
Copy link
Contributor

其实我个人觉得,可以直接去掉这一行。

从过往的 Git 记录来看,这是为了让稳定版用户知晓这一情况而临时写的。

现在用户一般都会用最新版本,这一条「注意事项」已经失去意义。

@TyMxy
Copy link
Contributor Author

TyMxy commented Nov 24, 2025 via email

@zkitefly
Copy link
Member

zkitefly commented Dec 3, 2025

确实,不过最终看仓库维护者的想法吧,如果他们也认为不需要的话,就删掉吧。

要不删掉吧

@TyMxy
Copy link
Contributor Author

TyMxy commented Dec 3, 2025

删除了

@TyMxy
Copy link
Contributor Author

TyMxy commented Dec 3, 2025

话说这篇文章中的“本文档在v3.4.208参考编写,有可能会的短时性的失效!”还需不需要了啊?目前看起来这篇文章的内容与最新版本并无差异。

@TyMxy
Copy link
Contributor Author

TyMxy commented Dec 3, 2025

我还注意到该文章的内容中有一句“注意: V3.4.205需把链接末尾的 / 去掉, 这个BUG已经在V3.4.206修复”,我认为现在用户都在用最新版本,这一条注意事项也已经失去了意义。

@burningtnt
Copy link
Member

此外,本 PR 有冲突

@TyMxy
Copy link
Contributor Author

TyMxy commented Dec 5, 2025

我没搞懂冲突在哪(

@3gf8jv4dv
Copy link
Contributor

我没搞懂冲突在哪(

点击页面中,评论输入框上方的「Resolve conflicts」,在这里可以解决。

@TyMxy
Copy link
Contributor Author

TyMxy commented Dec 5, 2025

这是,不能修改日期?
图片

@TyMxy
Copy link
Contributor Author

TyMxy commented Dec 5, 2025

你们如果能解决冲突的话,直接修改就行,我实在搞不懂awa

@3gf8jv4dv
Copy link
Contributor

这是,不能修改日期?

点击「Accept current change」即可,然后点击右上角的「Mark as resolved」。

@TyMxy
Copy link
Contributor Author

TyMxy commented Dec 5, 2025

冲突已解决

@3gf8jv4dv
Copy link
Contributor

3gf8jv4dv commented Dec 5, 2025

嘶,刚才看错了。应该点 Accept both change 的(

介意的话,你再交个 commit,把 excerpt: 使用本地皮肤文件、LittleSkin 或 Blessing Skin 设置离线皮肤 这句话加回去吧。

冲突是因为这段时间有 PR 添加了这句话。

@TyMxy
Copy link
Contributor Author

TyMxy commented Dec 5, 2025

额,好吧。

@burningtnt
Copy link
Member

burningtnt commented Dec 5, 2025

@3gf8jv4dv
Copy link
Contributor

这预览 URL 404 了。咋回事 🤔

@burningtnt
Copy link
Member

burningtnt commented Dec 6, 2025

@burningtnt
Copy link
Member

这预览 URL 404 了。咋回事 🤔

和另一个 PR 的关闭时自动预览清理功能撞车了,重新创建一份即可解决。介于 GitHub 的 concurrency 无法实现“排队”功能,完全修复该问题的成本较高,与获得的收益不成正比。

@burningtnt burningtnt closed this Dec 6, 2025
@burningtnt burningtnt reopened this Dec 6, 2025
@TyMxy
Copy link
Contributor Author

TyMxy commented Dec 6, 2025

已按照您的要求完成了修改

@burningtnt
Copy link
Member

burningtnt commented Dec 6, 2025

@TyMxy
Copy link
Contributor Author

TyMxy commented Dec 6, 2025

/preview create

@burningtnt
Copy link
Member

burningtnt commented Dec 6, 2025

@TyMxy
Copy link
Contributor Author

TyMxy commented Dec 6, 2025

我原来无法生成pr吗?awa

@TyMxy
Copy link
Contributor Author

TyMxy commented Dec 6, 2025

这两句似乎挤在一起了?
图片

@TyMxy
Copy link
Contributor Author

TyMxy commented Dec 6, 2025

这下可以了

@burningtnt
Copy link
Member

burningtnt commented Dec 6, 2025

@burningtnt
Copy link
Member

预览构建中,文本依然不能正确分行

@TyMxy
Copy link
Contributor Author

TyMxy commented Dec 6, 2025

这不是正常了吗?
图片

@burningtnt
Copy link
Member

这不是正常了吗?

奇怪了,当时我看的时候是异常的 :(

@burningtnt burningtnt merged commit ca00930 into HMCL-dev:main Dec 6, 2025
7 checks passed
@TyMxy
Copy link
Contributor Author

TyMxy commented Dec 6, 2025

话说这两句话之间要不要空一行?感觉观感上更好?

@TyMxy TyMxy deleted the patch-1 branch December 6, 2025 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants