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

Use ReadTheDocs theme for documentation #2723

Merged
merged 3 commits into from Apr 17, 2019
Merged

Use ReadTheDocs theme for documentation #2723

merged 3 commits into from Apr 17, 2019

Conversation

joschi
Copy link
Member

@joschi joschi commented Apr 6, 2019

Instead of hosting the Dropwizard documentation on GitHub pages and having to update the gh-pages branch (semi-) manually after each release, we can use https://readthedocs.org/ which will take care of building the documentation and serving it in multiple versions.

While the original Dropwizard theme could be used on ReadTheDocs, the official RTD theme providers some features which we would have to add to the original theme, such as:

  • Search function
  • Version selection
  • Automatic sitemap generation

@joschi joschi added this to the 2.0.0 milestone Apr 6, 2019
@joschi joschi self-assigned this Apr 6, 2019
@joschi
Copy link
Member Author

joschi commented Apr 6, 2019

Preview: https://dropwizard.readthedocs.io/en/rtd/

@dropwizard-committers When somebody wants to port the search bar and the version selection to Dropwizard's original theme, I'm happy to keep using it.

Copy link
Member

@jplock jplock left a comment

Choose a reason for hiding this comment

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

I like this a lot, great work

Copy link
Contributor

@nickbabcock nickbabcock left a comment

Choose a reason for hiding this comment

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

Aww yeah, looks very nice.

@pstackle
Copy link
Contributor

Overall, looks good, but one unfortunate downside I see is that the Dropwizard Configuration Reference page becomes much harder to use due to the layout being so narrow.

@joschi
Copy link
Member Author

joschi commented Apr 16, 2019

@pstackle I've made the theme a little bit wider to accommodate wide tables (although the configuration reference still doesn't fit).

@dropwizard-committers If you're okay with it, I'd merge this PR and release Dropwizard 2.0.0-rc1 in the next days.

The latest version of the docs created from this branch are at https://dropwizard.readthedocs.io/en/rtd/.

@joschi joschi merged commit ab42c8c into master Apr 17, 2019
@joschi joschi deleted the rtd branch April 17, 2019 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants