-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Remove Pulse library URL renaming in docs #13107
Remove Pulse library URL renaming in docs #13107
Conversation
One or more of the following people are relevant to this code:
|
If you're willing to cherry-pick to 1.2, that would help the docs team so that the 1.3 release is less stressful. No big deal otherwise. |
Pull Request Test Coverage Report for Build 10745064909Details
💛 - Coveralls |
Bump |
I don't have a great history of success building the docs locally (on my Mac) but running it on this branch I'm getting
Did it work on your Mac? |
Julien: try |
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.
I have managed to successfully build the docs in MacOS with tox -e docs-clean,docs
, @Cryoris. Preview looks good and links work.
(cherry picked from commit c5e9f8a)
(cherry picked from commit c5e9f8a) Co-authored-by: Eric Arellano <[email protected]>
The workaround from #9748 is no longer necessary because there is no conflict with capitalization. So, we can go back to using the default URL for these classes, such as
qiskit.pulse.library.Constant
.We will set up redirects in docs.quantum.ibm.com for this URL change.