-
Notifications
You must be signed in to change notification settings - Fork 854
Publish the rust-docs in GH pages for easier usage and access #1206
Comments
Hey guys, I wonder if this issue is up-to-date. If yes, I would love to have the chance to pick up this task. |
Hi @ryanycw, feel free to take this task! It is still valid! |
Hey guy, I've finished the basic manual doc generation. And I have two ideas in my mind to improve the process. One is to use a CI action to update the docs automatically. The second is to add a husky hook to update the docs in every commit. I prefer the second approach since it won't add any risk to the repository by only running on the local repository. With the first approach, I've done some experiment. And it requires permissions to push the commit of updating the docs with GitHub actions, which is riskier for the project. I want to ask for your opinions to see if there is anything I oversee and what your suggestions would be. |
Hi @ryanycw, We shouldn't check in the |
@ChihChengLiang Thank you for checking in on the progress! If I open a new branch and reproduce the work with a cleaner commit history, will it be easier for you to review it? I hope I can make your job easier :). |
Hi @ryanycw, please send a "Draft pull request" to https://github.com/privacy-scaling-explorations/zkevm-circuits. |
Hi @ChihChengLiang, thank you so much for the guidance. I've just sent the draft PR. Would love to hear your suggestions and comments. |
Here is the result of what the GitBook will look like. It turns out that the permission issue isn't related to the deployment with GitHub. And I've found the great work from the reth team. Update: I've given mdbook a shot, and it seems there is a format requirement to use that. So, I believe rust-docs is a more feasible solution at this moment. |
Hi @ryanycw, For mdbook, we have a documentation in mdbook style, too. It is https://github.com/privacy-scaling-explorations/zkevm-docs. |
Hi @ChihChengLiang, Thank you again for all the guidance and support! I've published the pull request and asked for feedback in the comments. In the meantime, can I ask for some directions on other issues that might be a good start to work on? I've gone through the good first issues, and most of them were assigned. I would love to work on #1194, but if there are other options I would like to keep #1194 to other first-time contributors since it would be a perfect start. |
### Description This pull request aims to run the cargo doc in CI to generate and publish the doc to GitHub Pages. ### Issue Link Solving #1206 ### Type of change - [ ] Bug fix (non-breaking change which fixes an issue) - [X] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) - [ ] This change requires a documentation update ### Contents The implementation is in _.github/workflows/github-pages.yml_. The goal is to generate the doc, upload the doc, and deploy the page. ### How Has This Been Tested? I have tested this in my [forked repository](https://github.com/ryanycw/zkevm-circuits/actions/workflows/github-pages.yml). And here is the [demo](https://ryanycw.github.io/zkevm-circuits/zkevm_circuits/) of what the result will look like. --------- Co-authored-by: Chih Cheng Liang <chihchengliang@gmail.com>
No description provided.
The text was updated successfully, but these errors were encountered: