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

Move to mkdocs #346

Draft
wants to merge 50 commits into
base: main
Choose a base branch
from
Draft

Move to mkdocs #346

wants to merge 50 commits into from

Conversation

ItsDrike
Copy link
Member

@ItsDrike ItsDrike commented Aug 1, 2024

This removes Sphinx in favor of mkdocs for the project documentation.

Warning

This is an experimental PR, it isn't at all guaranteed that the project really will end up switching from sphinx to mkdocs, however, mkdocs does provide a much better user experience in terms of the overall design than sphinx and solves many other issues. This PR serves as a test, to see whether such a transition would be viable for us.

One thing that may or may not be a significant obstacle is porting the API reference. All of our docstrings currently use rst syntax and I'm not sure if mkdocs can handle that, or if it would mean having to rewrite them. If we would need to rewrite, another important question would then be whether the new syntax would support as many things as the sphinx one did. Worst case, it would actually be possible to keep sphinx alongside mkdocs, with sphinx being purely for the API reference. This may actually be worth it as the UX benefit from mkdocs is massive.

  • Automatic deployment to GitHub pages
  • Versioned docs
  • Towncrier changelog integration
  • Info pages
    • Introduction
    • Installation category
      • Installation guide
      • Version guarantees
      • Changelog (released changes)
    • Community category
      • Code of Conduct
      • Attributions
      • Security Policy
      • License
    • Contributing category
      • Reporting a bug
      • Asking a question
      • Making a pull request
    • Contributing guidelines category
      • Index
      • Setting up the project
      • Style Guide
      • Docstring formatting directive
      • Type hinting
      • Slotscheck
      • Pre-commit
      • Changelog
      • Great commits
      • Unit Tests
      • Deprecations
    • Frequently Asked Questions
    • Usage guides
      • ...
    • Examples?
  • API Reference

@ItsDrike ItsDrike added a: documentation Related to project's documentation (comments, docstrings, docs) p: 2 - normal Normal priority t: rewrite Complete or partial rewrite of something labels Aug 1, 2024
@ItsDrike ItsDrike force-pushed the mkdocs branch 6 times, most recently from f9597f3 to 8288c8f Compare August 1, 2024 21:39
Copy link

github-actions bot commented Aug 1, 2024

PR Preview Action v1.4.7
🚀 Deployed preview to https://py-mine.github.io/mcproto/pr-preview/pr-346/
on branch gh-pages at 2024-08-09 09:24 UTC

@ItsDrike ItsDrike force-pushed the mkdocs branch 10 times, most recently from b7067cf to 288dba2 Compare August 2, 2024 21:24
@ItsDrike ItsDrike force-pushed the mkdocs branch 3 times, most recently from a56a2ca to 20230a0 Compare August 5, 2024 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a: documentation Related to project's documentation (comments, docstrings, docs) p: 2 - normal Normal priority t: rewrite Complete or partial rewrite of something
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant