| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
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
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| Expand Up | @@ -101,12 +101,12 @@ C API Tests | |||||
| Tests for the public C API live in the ``_testcapi`` module. | ||||||
| Functions named ``test_*`` are used as tests directly. | ||||||
| Tests that need Python code (or are just easier to partially write in Python) | ||||||
| live in ``Lib/test``, mainly in :file:`Lib/test/test_capi.py`. | ||||||
| live in ``Lib/test``, mainly in :cpy-file:`Lib/test/test_capi`. | ||||||
|
Comment thread
Copy link
Copy Markdown
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Qualitytest_capi is now a dir.
Sorry, something went wrong.
All reactions
Comment thread
Copy link
Copy Markdown
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low QualityWe could also link things that aren't already :file:, like Lib/test:
Suggested change
There's a lot more that could be linked on this page too, especially the first few sections. But let's keep the scope of this PR to replacing the old :file: with the new :cpy-file:.
Sorry, something went wrong.
ezio-melotti and CAM-Gerlach reacted with thumbs up emoji
All reactions
Copy link
Copy Markdown
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low QualityAgreed on keeping the PR focused. We should also try to create links only when/where they are useful, but there are definitely more places where :cpy-file: can be used.
Sorry, something went wrong.
hugovk and CAM-Gerlach reacted with thumbs up emoji
All reactions
Copy link
Copy Markdown
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low QualityThat SGTM
Sorry, something went wrong.
All reactions
|
||||||
|
|
||||||
| Due to its size, the ``_testcapi`` module is defined in several source | ||||||
| files. | ||||||
| To add a new set of tests (or extract a set out of the monolithic | ||||||
| :file:`Modules/_testcapimodule.c`): | ||||||
| :cpy-file:`Modules/_testcapimodule.c`): | ||||||
|
|
||||||
| - Create a C file named ``Modules/_testcapi/yourfeature.c`` | ||||||
|
|
||||||
| Expand Down | ||||||
| Back | FazBrowse Home | New Git URL |
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.
Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low QualityIf we add ~ support to :cpy-file: we could hide the dir.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.