Skip to content

Fix sphinx doc#26

Merged
nmellado merged 30 commits into
mainfrom
fixSphinxDoc
May 14, 2024
Merged

Fix sphinx doc#26
nmellado merged 30 commits into
mainfrom
fixSphinxDoc

Conversation

@hiergaut

Copy link
Copy Markdown
Contributor

Make sphinx documentation without compilation error

@hiergaut hiergaut marked this pull request as draft March 26, 2024 15:22
@hiergaut

Copy link
Copy Markdown
Contributor Author

waiting for #21 merged

@hiergaut hiergaut mentioned this pull request Mar 28, 2024
@hiergaut hiergaut added documentation Improvements or additions to documentation CI Continuous integration (yml files) labels Apr 5, 2024
@hiergaut hiergaut marked this pull request as ready for review April 8, 2024 12:18
@hiergaut

hiergaut commented Apr 8, 2024

Copy link
Copy Markdown
Contributor Author

Another PR will clean the doc, this pr only fix compilation, assume there are some bad urls in the doc.

Comment thread .github/workflows/deployDoc.yml
Comment thread .github/workflows/deployDoc.yml Outdated
@nmellado

Copy link
Copy Markdown
Contributor

Let me know when I can review

@hiergaut

Copy link
Copy Markdown
Contributor Author

I had some problem (with xml path in server path) to build in build folder, I will fix this probably tomorow.

@hiergaut hiergaut mentioned this pull request Apr 10, 2024
@nmellado nmellado marked this pull request as draft April 10, 2024 14:26
@hiergaut hiergaut force-pushed the fixSphinxDoc branch 2 times, most recently from fe6d01d to 3378411 Compare April 22, 2024 11:52
@hiergaut

Copy link
Copy Markdown
Contributor Author

@nmellado ready for review if tests passed.

@hiergaut hiergaut marked this pull request as ready for review April 22, 2024 12:34
@hiergaut hiergaut mentioned this pull request Apr 22, 2024

@nmellado nmellado left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Except two minor comments, looks fine !

Comment thread doc/sphinx/Doxyfile.in Outdated
Comment thread doc/sphinx/Doxyfile.in Outdated
@hiergaut hiergaut mentioned this pull request Apr 22, 2024
7 tasks
@hiergaut

Copy link
Copy Markdown
Contributor Author

I tested the deploy.

@hiergaut

Copy link
Copy Markdown
Contributor Author

An issue #47 was added to make all build files into build dir.

@hiergaut hiergaut added the TOPULL #2 Pull request to be done secondly label May 7, 2024
@nmellado nmellado merged commit a2b956a into main May 14, 2024
@nmellado nmellado deleted the fixSphinxDoc branch May 14, 2024 09:08
@nmellado nmellado removed the TOPULL #2 Pull request to be done secondly label May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continuous integration (yml files) documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants