-
Notifications
You must be signed in to change notification settings - Fork 2
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: DynamicScrapper #281
fix: DynamicScrapper #281
Conversation
Thanks for opening a Pull Request. If you want to perform a review write a comment saying: @ansys-reviewer-bot review |
Are you sure is it a cache issue? The doc run that deployed didnt use the cache. https://github.com/ansys/pyansys-math/actions/runs/7992151629/job/21825019425#step:9:32 |
For some reasons, when using |
Building documentation with Building documentation manually: I will investigate why using |
Disabling the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's use proper boolean env vars - apart from that LGTM
So, there is an issue with the interactive documentation when using the link checker from Sphinx, is that the case? Out of curiosity, when you tried manually, you were using the link checker too? We might have to add a regex for the links from the interactive docs to |
I'm not sure, but adding this to the linkcheck_ignore = [r'*.vtksz'] |
I agree with @AlejandroFernandezLuces - let's ignore the files where this happens, or the extensions to the files involved. |
Co-authored-by: Roberto Pastor Muela <[email protected]>
Thank you for your feedbacks @AlejandroFernandezLuces and @RobPasMue. |
When merging #197, the following error appeared in the dev documentation:
This PR is to fix this rendering error.