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

Add Automatic Build and Deployment of Doxygen Documentation to GitHub Pages #817

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ngpaladi
Copy link

@ngpaladi ngpaladi commented Feb 2, 2025

An example of a functional deployment is available at: https://www.uncommonmodels.com/openmrn/

Please note that the Pages deployment source must be set to "GitHub Actions" in the repository settings.

@bakerstu
Copy link
Owner

bakerstu commented Feb 2, 2025

@ngpaladi This is an excellent contribution. I want to run this by some of the other frequent contributors.

...now of course we should also spend some effort updating the Doxygen to be a bit more up to date.

@balazsracz
Copy link
Collaborator

I'm not super familiar with actions. This looks good to me and we should have it.
Stuart, I think you are the only one who can make the change in the settings.

@atanisoft
Copy link
Collaborator

I've used a similar set of actions but instead had:

- name: Setup Pages
        id: pages
        uses: actions/configure-pages@v3

included as part of it to retrieve the page_url string. I believe this should work provided the project settings are updated accordingly.

@ngpaladi
Copy link
Author

ngpaladi commented Feb 3, 2025

Thank you all for considering adding this! I hope this will ultimately make it easier for people to get started.

I've used a similar set of actions but instead had:

- name: Setup Pages
        id: pages
        uses: actions/configure-pages@v3

included as part of it to retrieve the page_url string. I believe this should work provided the project settings are updated accordingly.

I believe this step is only necessary if you need to force enable Pages without making the aforementioned repository settings changes, or if you need to grab the final URL as an input for running Doxygen. The former does not occur by default and likely would create permission issues.

@atanisoft
Copy link
Collaborator

if you need to grab the final URL as an input for running Doxygen.

In my case I'm using Jekyll to transform md into html files and it uses the url as an input, the file was generated as part of configuring pages the first time via repository settings. I think in this case it likely is fine to skip it but we should test it as part of enabling pages support and adjust if required.

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.

4 participants