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

Update README Adding Sort by star system 加入了根据星级分类菜谱系统 #1327

Closed
wants to merge 45 commits into from

Conversation

YufeiJ1ao
Copy link
Contributor

@YufeiJ1ao YufeiJ1ao commented Jun 5, 2024

According to issue #1326
根据 #1326 提出的
Adding Sort by star system in README.md
在README.md末尾加入了根据星级分类的菜谱系统方便快速查看

Adding starSystem.py when there's update of the recipe run this file to write the new recipe into the star system
https://github.com/YufeiJ1ao/HowToCook/blob/master/starSystem.py
加入starSystem.py当菜谱有更新时可以运行这个文件把新的菜谱写入星级系统里

注意

Pull Request 提交后,预计 1 分钟内将会得到自动化格式检查的结果。只有通过自动化检查的 Pull Request 才会被人工审核。

  • 请确保此 Pull Request 能够通过自动化代码检查。

签署

必须签署下面的对话框才能开始审核。

HowToCook 仓库采用了 The Unlicense 协议

菜谱在签入前,必须确保其可以直接声明进入 "公共领域"(public domain)。这意味着一旦合并后,任何人都可以自由复制,修改,发布,使用,编译,出售或以菜谱的形式或菜的形式分发,无论是出于商业目的还是非商目的,以及任何手段。

  • 我确保了我的内容不涉及版权内容,并且授权它 The Unlicense 协议。

@YufeiJ1ao
Copy link
Contributor Author

其实这个功能您觉得有必要吗, 如果您觉得可以要我可以把这个放在一个新文件夹里面
In fact, do you think this feature is necessary? if you think so I can put this in a new folder.

@Anduin2017
Copy link
Owner

记得在docker编译的时候执行这个。

Copy link
Owner

@Anduin2017 Anduin2017 left a comment

Choose a reason for hiding this comment

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

另外我建议使用 node 实现而不是 python。因为目前 python 在这个仓库里是用于构建最终的网站的。readme的构建是完全基于 npm run build 并运行了一段node脚本实现的。

你可以使用 chatgpt 将你的代码翻译成 node 的版本,并签入:

https://github.com/Anduin2017/HowToCook/blob/master/.github/readme-generate.js

使其生成出来带星级索引的 readme.md 和 mkdocs.yml。

@YufeiJ1ao
Copy link
Contributor Author

好的 那我先close这个pull request

@YufeiJ1ao YufeiJ1ao closed this Jun 5, 2024
@YufeiJ1ao YufeiJ1ao mentioned this pull request Jun 5, 2024
2 tasks
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.

2 participants