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

Convert docs/ folder to mdBook #4684

Merged
merged 4 commits into from
Jun 3, 2024
Merged

Conversation

leo60228
Copy link

@leo60228 leo60228 commented May 31, 2024

This builds web documentation from the docs/ folder using mdBook, a language-agnostic documentation tool developed by the Rust programming language project.

Description

To avoid changing the existing directory structure, a Python preprocessor is used as a simple way to rewrite links. This is a slightly hacky solution, but seemed better than the alternatives.

mdBook only implements a subset of GitHub Flavored Markdown features, so the large number of autolinks in the changelogs needed to be replaced with standard Markdown bare links (https://example.com<https://example.com>). Now done as part of fix_links.py.

The documentation generated by this PR is currently deployed at https://leo60228.github.io/pokeemerald.

Doing this was suggested by @AsparagusEduardo on Discord:

I think the first thing that needs to be researched is how to set up a documentation site like how porymap does it, so it's kept within the repo itself and can be updated more easily to the fluctuating changes that happen. I'll try to set that up for master

Note that porymap uses Sphinx rather than mdBook. From what I've seen, Sphinx is much less popular than mdBook for new projects.

Images

Screenshot

Discord contact info

leo60228

@AsparagusEduardo AsparagusEduardo merged commit e925c51 into rh-hideout:master Jun 3, 2024
1 check passed
AsparagusEduardo added a commit that referenced this pull request Jun 3, 2024
@leo60228 leo60228 mentioned this pull request Jun 3, 2024
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