python / cpython

The Python programming language
https://www.python.org
Other
63.72k stars 30.53k forks source link

gh-127183: Add `_ctypes.CopyComPointer` tests #127184

Closed junkmd closed 4 days ago

junkmd commented 6 days ago

I would like to backport this to 3.12 and 3.13 as well. This is internal-only, so I donโ€™t think it needs a NEWS entry.

junkmd commented 4 days ago

Would you be willing to add some documentation for it as well?

That's my intention, but as discussed in gh-126686, I believe publicizing it is also necessary for proper documentation.

Before moving forward with documentation or publicizing, I want to backport this test to 3.13 and 3.12, where CopyComPointer, albeit private, already exists. Therefore, I aim to keep this PR in a state where it can be merged as-is when miss-islington backports it.

miss-islington-app[bot] commented 4 days ago

Thanks @junkmd for the PR, and @encukou for merging it ๐ŸŒฎ๐ŸŽ‰.. I'm working now to backport this PR to: 3.12, 3.13. ๐Ÿ๐Ÿ’โ›๐Ÿค–

bedevere-app[bot] commented 4 days ago

GH-127251 is a backport of this pull request to the 3.13 branch.

bedevere-app[bot] commented 4 days ago

GH-127252 is a backport of this pull request to the 3.12 branch.

encukou commented 4 days ago

Makes sense. Thank you for the test!