Skip to content

Conversation

Baertig
Copy link

@Baertig Baertig commented Mar 13, 2025

Contribution description

Currently the preview documentation (e.g. here) is build with another doxygen version then the website (https://doc.riot-os.org/).
To deploy the website this Dockerfile is used: OlegHahm/docker-doxygen, but to build the preview we use the riot/murdock-scripts:latest docker image.
I think it would be best if both are build with the same image, so that the doxygen version can be maintained in only one place. Therefor I added a github workflow, that uses the docker-doxygen image to build the documentation and publish it as an artifact.

Limitations: currently you can only preview the documentation locally, by downloading and unzipping the artifact.
I am trying to figure out a good way to provide a preview, that one can view directly in the browser without downloading. (that's why it is in draft)

Testing procedure

You can check how it would look like in this example PR in my fork: Baertig#1
(click on checks and then on preview-documentation)

Issues/PRs references

Fixes #21106

gh-actions: change compression level to 0
@github-actions github-actions bot added the Area: CI Area: Continuous Integration of RIOT components label Mar 13, 2025
@Baertig Baertig force-pushed the ci/add_create_doc_workflow branch from cd419d5 to 93c3256 Compare March 14, 2025 14:45
@crasbe
Copy link
Contributor

crasbe commented Mar 14, 2025

After thinking about this a lot, I had the idea to integrate Doxygen as a tool in the RIOT dist/tools folder. This way we could truly have one single spot where the Doxygen version is maintained.

To make it a bit easier to understand I created this PR: #21300

This is very much up for debate, but I thought it would be easier to create the PR to picture what the idea is.

@Baertig Baertig closed this Apr 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: CI Area: Continuous Integration of RIOT components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Build System: Update Doxygen to >=1.10.0 to fix URLs in Headlines
2 participants