-
-
Notifications
You must be signed in to change notification settings - Fork 409
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
Fixing the documentation bug #1527
Fixing the documentation bug #1527
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1527 +/- ##
=======================================
Coverage 67.57% 67.57%
=======================================
Files 68 68
Lines 6103 6100 -3
=======================================
- Hits 4124 4122 -2
+ Misses 1979 1978 -1
Continue to review full report at Codecov.
|
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.
Thanks for figuring it out @atharva-2001 - excellent job!
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.
Looks good. Please watch that issue to get news about it. We do not want to pin so many dependencies.
Sure. I will keep a watch on it. |
This reverts commit 09f992f.
The PR is an attempt to fix the documentation bug, mentioned in #1522 and #1523
Description
The sphinx-rtd-theme is incompatible with docutil v0.17, and downgrading it solves the problem. This issue is tracked here.
Motivation and Context
resolves #1523
resolves #1522
How Has This Been Tested?
I have tested this in my documentation.
Screenshots (if appropriate):
Before:
After:
Types of changes
Checklist: