-
Notifications
You must be signed in to change notification settings - Fork 2
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
[Feature] Switch from sphinx to mkdocs #57
Comments
@zaanposni Could you add the link (https://vvspy.readthedocs.io/en/latest/) to repository description (in the link section of course)? I think it would be a bit more convenient. |
When switching from sphinx to Mkdocs it would also be required to update all doc strings because they are not compatible 100% currently. |
I agree that mkdocs looks more modern and should be used.
Would be good to review/rework those doc strings anyway 😄 |
@felixhoffmnn do you want to work on this issue? |
Sounds good. Yeah, you can assign me 😃 I was slightly over-motivated and started working on #55, #56, and #57. You can look through the current progress here: https://github.com/felixhoffmnn/vvspy. If there appears anything that is not wanted or not in this project's scope, please comment on the regarding commit. |
Hi @felixhoffmnn as you may have guessed I am currently a bit busy ^.^ The only thing I dont like is the use of jupyter notebooks as examples and the new logging framework as those things introduce unnecessary third party dependencies. I would have to go through your changes more carefully once you feel like they are ready to review. Maybe you should split them up in smaller PRs too ^.^ |
Again, I can work on issues too if you want me to. No need to do everything on your own. If you want: Feel free to add me on discord for easier and direct communication: zaanposni#9295 |
wont be fixed for now |
Summarize the feature
In my opinion, Mkdocs is a more modern and easier approach to documentation in python.
Mkdocs is also compatible with readthedocs.
Example of the feature
FastAPI, mkdocs-material
Additional context
None
The text was updated successfully, but these errors were encountered: