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

Fix the docs build on read the docs #17

Merged
merged 6 commits into from
Nov 24, 2023
Merged

Fix the docs build on read the docs #17

merged 6 commits into from
Nov 24, 2023

Conversation

clemens-fricke
Copy link
Member

Currently is the build on read the docs broken since it does not show any documentation of the source code.

Copy link
Member

@danielwolff1 danielwolff1 left a comment

Choose a reason for hiding this comment

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

Thanks for addressing this issue 👍

I just had a few minor remarks (mostly spelling/grammar), once the changes are accepted, we can approve this PR and merge it!

README.md Outdated Show resolved Hide resolved
docs/source/index.rst Outdated Show resolved Hide resolved
docs/source/index.rst Outdated Show resolved Hide resolved
@clemens-fricke
Copy link
Member Author

@danielwolff1 Thank you for the quick review. I have added the changes.

@danielwolff1
Copy link
Member

Thanks for adopting the requested changes so quickly! From my point of view this can be merged now! 🚀

@clemens-fricke clemens-fricke merged commit a139d65 into main Nov 24, 2023
1 check passed
@clemens-fricke clemens-fricke deleted the fix-rtd-build branch November 24, 2023 10:11
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.

2 participants