Skip to content

[smart_holder] Debugging test_cross_module_exception_translator issue #4053

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

Closed
wants to merge 8 commits into from

Conversation

rwgk
Copy link
Collaborator

@rwgk rwgk commented Jul 8, 2022

Description

IGNORE this PR! Debugging came to a conclusion under PR #4054.

Suggested changelog entry:

rwgk added 6 commits July 7, 2022 20:51
To probe environment/toolchain/platform-specific behavior under the exact same conditions as normal tests.

(An earlier version of this code was used to inform PR pybind#4043.)
…and CUDA `test_cross_module_exception_translator` failures).
…s action un-helpfully removed the added noqa).
…files, only setting the module docstring and doing nothing else.
@rwgk rwgk closed this Jul 8, 2022
@rwgk
Copy link
Collaborator Author

rwgk commented Jul 8, 2022

Forgot to select the smart_holder branch.

@rwgk rwgk reopened this Jul 8, 2022
@rwgk rwgk changed the base branch from master to smart_holder July 8, 2022 19:17
@rwgk rwgk closed this Jul 8, 2022
@rwgk rwgk reopened this Jul 8, 2022
@rwgk
Copy link
Collaborator Author

rwgk commented Jul 8, 2022

It let me change the target branch only on the open PR. Oh well.

rwgk added a commit to rwgk/pybind11 that referenced this pull request Jul 8, 2022
@rwgk rwgk closed this Jul 8, 2022
@rwgk rwgk deleted the debugging branch July 8, 2022 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant