-
Notifications
You must be signed in to change notification settings - Fork 22
修正内容 #358
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
修正内容 #358
Conversation
|
另外我想问一下,对于文档中最低版本的声明,应该写最低正式版还是最低测试版还是两个都写? |
|
其实我个人觉得,可以直接去掉这一行。 从过往的 Git 记录来看,这是为了让稳定版用户知晓这一情况而临时写的。 现在用户一般都会用最新版本,这一条「注意事项」已经失去意义。 |
|
确实,不过最终看仓库维护者的想法吧,如果他们也认为不需要的话,就删掉吧。
|
要不删掉吧 |
|
删除了 |
|
话说这篇文章中的“本文档在v3.4.208参考编写,有可能会的短时性的失效!”还需不需要了啊?目前看起来这篇文章的内容与最新版本并无差异。 |
|
我还注意到该文章的内容中有一句“注意: V3.4.205需把链接末尾的 / 去掉, 这个BUG已经在V3.4.206修复”,我认为现在用户都在用最新版本,这一条注意事项也已经失去了意义。 |
|
此外,本 PR 有冲突 |
|
我没搞懂冲突在哪( |
点击页面中,评论输入框上方的「Resolve conflicts」,在这里可以解决。 |
|
你们如果能解决冲突的话,直接修改就行,我实在搞不懂awa |
点击「Accept current change」即可,然后点击右上角的「Mark as resolved」。 |
|
冲突已解决 |
|
嘶,刚才看错了。应该点 Accept both change 的( 介意的话,你再交个 commit,把 冲突是因为这段时间有 PR 添加了这句话。 |
|
额,好吧。 |
|
这预览 URL 404 了。咋回事 🤔 |
和另一个 PR 的关闭时自动预览清理功能撞车了,重新创建一份即可解决。介于 GitHub 的 concurrency 无法实现“排队”功能,完全修复该问题的成本较高,与获得的收益不成正比。 |
|
已按照您的要求完成了修改 |
|
/preview create |
|
我原来无法生成pr吗?awa |
|
这下可以了 |
|
预览构建中,文本依然不能正确分行 |
奇怪了,当时我看的时候是异常的 :( |
|
话说这两句话之间要不要空一行?感觉观感上更好? |



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