Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix issues in the documentation #146

Merged
merged 9 commits into from
May 24, 2023
Merged

Conversation

AngelFP
Copy link
Member

@AngelFP AngelFP commented May 23, 2023

Fixes a few issues in the documentation, including the incorrect rendering of math expressions (#145), broken examples, and other minor problems.

To prevent these issues in the future, the documentation build will now fail if any warnings are raised.

@AngelFP AngelFP requested review from ax3l and RemiLehe May 23, 2023 09:03
Copy link
Contributor

@MaxThevenet MaxThevenet left a comment

Choose a reason for hiding this comment

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

Great, thanks for the cleaning! Let's see how this fail_on_warning: true works in practice.

@MaxThevenet
Copy link
Contributor

Could you resolve merge conflicts?

@AngelFP
Copy link
Member Author

AngelFP commented May 24, 2023

Thanks for checking! It should be ready to merge now :)

@MaxThevenet MaxThevenet merged commit d965f82 into LASY-org:development May 24, 2023
@AngelFP AngelFP deleted the fix_docs branch May 24, 2023 11:41
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants