Skip to content

Conversation

@kurannn
Copy link
Contributor

@kurannn kurannn commented Apr 7, 2025

Adding documentation

@kurannn kurannn requested a review from mmpagani April 7, 2025 13:59
@kurannn kurannn requested a review from mmpagani April 8, 2025 09:35
Copy link
Member

@mmpagani mmpagani left a comment

Choose a reason for hiding this comment

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

Sorry, two additional comments then I think we are ready to merge.


html_theme = 'sphinx_rtd_theme'
html_static_path = ['_static']
#html_static_path = ['_static']
Copy link
Member

Choose a reason for hiding this comment

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

Why are you changing this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I commented out html_static_path because the _static folder doesn't exist in the repository. This avoids Sphinx warnings during the documentation build. If we add custom static files in the future, we can uncomment it again.

@kurannn kurannn requested a review from mmpagani April 8, 2025 10:01
@kurannn kurannn merged commit 2e4cc44 into main Jul 2, 2025
2 checks passed
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.

3 participants