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.

Sign up for GitHub

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

Jump to bottom

1.Modify the layout and some contents;2.Change the license img source to the url. #12

Merged
merged 7 commits into from
Apr 1, 2023
Merged

1.Modify the layout and some contents;2.Change the license img source to the url. #12

merged 7 commits into from
Apr 1, 2023

Conversation

Tenktau
Copy link
Contributor

@Tenktau Tenktau commented Mar 30, 2023

No description provided.

@Tenktau
Copy link
Contributor Author

Tenktau commented Mar 30, 2023

Deploy失败我在自己的仓库也遇到过，原因应该是Git Action的Bot没有仓库的Write权限。

wsong83
wsong83 reviewed Mar 30, 2023
View reviewed changes
docs/ch3/reference.md
Copy link
Member

Choose a reason for hiding this comment

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

I dont think it is a good idea to remove stuff that might be useful in the future? I would like this file to be restored before accepting the PR.

@wsong83
Copy link
Member

wsong83 commented Mar 30, 2023

check失败不是什么问题。但是问题是不要轻易删除暂时看起来没用的内容。关于文件结构的修改暂时先这样吧。

@wsong83
Copy link
Member

wsong83 commented Mar 30, 2023

The original docs/ch3/reference.md was not empty?

@wsong83 wsong83 merged commit 1de8e0d into cnrv:master Apr 1, 2023
@wsong83
Copy link
Member

wsong83 commented Apr 1, 2023

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wsong83 wsong83 wsong83 left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Tenktau @wsong83