gh-151950: Fix dangling class reference in pickletools docs#152209
gh-151950: Fix dangling class reference in pickletools docs#152209BHUVANSH855 wants to merge 1 commit into
Conversation
BHUVANSH855
force-pushed
the
gh-151950-pickletools
branch
2 times, most recently
from
1193a9c to
acc1683
Compare
BHUVANSH855
requested review from
AA-Turner,
StanFromIreland and
hugovk
as code owners
|
A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated. Once you have made the requested changes, please leave a comment on this pull request containing the phrase |
BHUVANSH855
force-pushed
the
gh-151950-pickletools
branch
from
acc1683 to
c17a831
Compare
Documentation build overview
92 files changed ·
|
|
I have made the requested changes; please review again. |
|
Thanks for making the requested changes! @StanFromIreland: please review the changes made to this pull request. |
| opcode's argument; *pos* is the position at which this opcode is located. | ||
| *pickle* can be a string or a file-like object. | ||
|
|
||
| .. class:: OpcodeInfo |
There was a problem hiding this comment.
This isn't in pickletools.__all__ so if we're documenting it we should add it there, and also ArgumentDescriptor and StackObject. They are technically already public per PEP 387, and we do return them, so I think we should expose them properly.
BHUVANSH855
force-pushed
the
gh-151950-pickletools
branch
from
c17a831 to
c819cac
Compare
2 participants
Issue
gh-151950
Description
This PR fixes a dangling Sphinx class reference in
Doc/library/pickletools.rst.The reference to
OpcodeInfopointed to an undocumented internal class, producing a Sphinx warning. This change replaces the cross-reference with:class:!OpcodeInfo`` to preserve the formatting while suppressing the unresolved reference.