-
-
Notifications
You must be signed in to change notification settings - Fork 31.9k
bpo-29455: Mention coverage.py in trace module documentation #261
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
Doc/library/trace.rst
Outdated
@@ -13,6 +13,10 @@ annotated statement coverage listings, print caller/callee relationships and | |||
list functions executed during a program run. It can be used in another program | |||
or from the command line. | |||
|
|||
.. seealso:: `Coverage.py <https://coverage.readthedocs.io/>`_, which is one |
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.
Looking at the rest of the documentation is seem like most .. seealso::
are alone on their lines with the paragraph on the next one.
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.
Doc/library/trace.rst
Outdated
.. seealso:: | ||
|
||
`Coverage.py <https://coverage.readthedocs.io/>`_ | ||
A popular third-party coverage tools that provides very nice HTML |
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.
tools -> tool?
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.
"very nice" can be dropped.
Doc/library/trace.rst
Outdated
|
||
`Coverage.py <https://coverage.readthedocs.io/>`_ | ||
A popular third-party coverage tools that provides very nice HTML | ||
output along with advanced features such as branch coverage. |
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.
advanced -> more advanced?
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.
LGTM, thanks! The issue is assigned to @brettcannon so I won't merge this.
) (cherry picked from commit 5dfccb0)
Move the controlling macro SLP_WITH_FRAME_REF_DEBUG to stackless.h. Make the code independent from Py_TRACE_REFS.
This pull request fixes bpo-29455.