Skip to content
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

[MOVED] Add pybind11 IWYU pragmas (for clangd Include Cleaner). #30113

Closed
wants to merge 35 commits into from

Conversation

rwgk
Copy link
Contributor

@rwgk rwgk commented Apr 1, 2024

Description

[Originally under #30073][Continued under #30152]

This PR is the source of truth for google3_iwyu_pragmas.patch

Merging and upstreaming this PR is deferred until the clangd Include Cleaner supports removing the "third_party/pybind11/include/" path name parts.

For easy future reference: commit 9e5f44d here was submitted as cl/577013593.

Suggested changelog entry:

Ralf W. Grosse-Kunstleve added 15 commits July 28, 2024 18:37
…art_holder_from_shared_ptr()`, to achieve successful building. (Testing aborts with a segfault in test_class_release_gil_before_calling_cpp_dtor.py)
…m smart_holder_type_casters.h to type_caster_base.h

Resolve all `PYBIND11K_MERGE_SH_AFTER_PR5257_WIP` in test_return_value_policy_override.py (in other words, restore the original complete tests).
…_casters.h to type_caster_base.h

Restore the original, complete test_class_sh_void_ptr_capsule.py
rwgk pushed a commit to rwgk/pybind11 that referenced this pull request Aug 26, 2024
@rwgk rwgk closed this Aug 26, 2024
@rwgk rwgk deleted the iwyu_pragmas_pybind11k branch August 26, 2024 20:49
@rwgk rwgk changed the title Add pybind11 IWYU pragmas (for clangd Include Cleaner). [MOVED] Add pybind11 IWYU pragmas (for clangd Include Cleaner). Aug 26, 2024
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.

2 participants