-
Notifications
You must be signed in to change notification settings - Fork 23
Ans melanie/op doc custom #2409
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
Conversation
…- moved toc.yml one level up
❌ 3 Tests Failed:
View the top 3 failed test(s) by shortest run time
To view more test analytics, go to the Test Analytics Dashboard |
|
The documentation for this pull request will be available at https://dpf.docs.pyansys.com/pull/2409. Please allow some time for the documentation to be deployed. |
|
Hey @AnsMelanie, I went ahead and made a few changes to ensure everything looks good on my end - 6b5e317 has the details. I essentially just removed a noisy print statement and ensure that the necessary filepaths were committed to the repo, and ensured the templates were where the script expects them to be. Looks good from my end now. Feel free to merge |
|
Hi @AnsMelanie I do not see the version or changelog in the |
|
I see the old template. I don't understand what happened. I'll check on Monday. |
|
@AnsMelanie this is definitely something I did, I can take a look tomorrow. |
Many changes since the initial review, it is not a valid approval any longer
0526940 to
6b7e053
Compare
|
Now dealt with in #2423 |
Hi @ansMHanmer,
Thanks for the review!