-
Notifications
You must be signed in to change notification settings - Fork 40
fix: update template by mdbook v0.4.52 #85
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
fix: update template by mdbook v0.4.52 #85
Conversation
mdbook v0.4.41 后修改了加载侧边栏目录的逻辑,需要同步修改模板。目前模板与部署时使用的 mdbook 版本不匹配导致侧边栏链接错误。通过 mdbook init --theme 生成新模板替换目前模板。 link: rust-lang/mdBook#2414
|
fixes #86 |
|
Fixes要写在PR正文或者提交消息里面才有效果 |
哭 |
RadxaYuntian
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
另外也要把src/share/rsdk/infra-package里面的docs workflow和devenv.lock更新,去使用devenv提供的mdbook
6ba6ba9 to
64f7872
Compare
64f7872 to
d4c28ee
Compare
RadxaYuntian
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
不是说要用devenv的环境来配置mdbook,不要在workflow里面安装了么
https://devenv.sh/integrations/github-actions/#run-multiple-commands
|
另外如果devenv的安装有问题的话,可以参考这里在我们devcontainer里面运行,这里面会预装devenv |
搞定了,没注意根根目录也配了个,只改了 rsdk/src/share/rsdk/infra-package 里面的 |
|
根目录的是rsdk自己实际使用的,usr/share里面是用于更新其他仓库的模板 |
1 |
d4c28ee to
6e07b1a
Compare
指定 mdbook 的版本, 避免后续 mdbook 更新版本与模板不匹配引入错误, 本地版本与部署版本保持一致, 方便发现问题. workflows 使用 devenv 运行运行环境.
6e07b1a to
8f6b6ab
Compare
1.mdbook v0.4.41 后修改了加载侧边栏目录的逻辑, 需要同步修改模板. 目前模板与部署时使用的 mdbook 版本不匹配导致侧边栏链接错误. 通过 mdbook init --theme 生成新模板替换目前模板.
2.giscus 客户端和语言切换按钮单独提交, 方便比对与模板的差异.
3.git restore 操作应该根据 po 文件的 PO-Revision-Date 字段, POT-Creation-Date 字段是创建时的时间, 后续更新 po 文件应该手动或者自动化工具更新 PO-Revision-Date 字段, 保证 git restore 还原 docs 文件夹到指定时间.
4.指定 mdbook 的版本, 避免后续 mdbook 更新版本与模板不匹配引入错误, 本地版本与部署版本保持一致, 方便发现问题.
link: rust-lang/mdBook#2414
fix #86