Skip to content

✨ Added Doxygen Doxyfile For Automated Documentation#49

Merged
liam-teale merged 40 commits intoZestCommunity:mainfrom
leoabubucker:main
Mar 4, 2025
Merged

✨ Added Doxygen Doxyfile For Automated Documentation#49
liam-teale merged 40 commits intoZestCommunity:mainfrom
leoabubucker:main

Conversation

@leoabubucker
Copy link
Contributor

@leoabubucker leoabubucker commented Mar 4, 2025

Overview

Adds Doxyfile in /docs to automatically generate documentation website for repository. This PR DOES NOT include the html files for documentation website as that is in a separate branch so a separate PR request.

Motivation

Public, and easy-to-access, documentation is needed for an open-source framework

References (optional)

Part of a fix that resolves #25

Implementation Details (optional)

See documentation website (the GitHub Page on my fork): ZestCode

Does not address automatic updating (needs to be manually re-deployed). Future PR request will introduce automatic updating.

It is recommended to have the website content in a separate branch, thus I have the website content in a gh-pages branch (what GitHub recommends it's named). I don't see a way to do a PR that creates a base gh-pages branch that I can merge my gh-pages branch to. Please advise.

Test Plan:

  • Test that addition of Doxyfile does not break other features

leoabubucker and others added 30 commits March 3, 2025 14:38
@liam-teale
Copy link
Member

Don't worry about GitHub pages. I like using readthedocs.io instead, as it will build the documentation for us.

Copy link
Member

@liam-teale liam-teale left a comment

Choose a reason for hiding this comment

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

LGTM

@liam-teale liam-teale merged commit 54d8296 into ZestCommunity:main Mar 4, 2025
@leoabubucker
Copy link
Contributor Author

Changing to using readthedocs.io which does not require a separate dedicated branch and is cleaner overall. Good idea! Testing website here, will make a PR when I work out the formatting better: https://zestcode.readthedocs.io/en/latest/

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.

📝 Documentation Hosting

3 participants