Skip to content

Splitting out include/pybind11/detail/pragma_warning_block.h - #2988

Merged
rwgk merged 3 commits into
pybind:smart_holderfrom
rwgk:sh_pragma_warning_block_h
May 4, 2021
Merged

Splitting out include/pybind11/detail/pragma_warning_block.h#2988
rwgk merged 3 commits into
pybind:smart_holderfrom
rwgk:sh_pragma_warning_block_h

Conversation

@rwgk

@rwgkrwgk commented May 1, 2021

Copy link
Copy Markdown
Collaborator

Alternative to moving the pragma block to common.h. This way the code organization on master can stay exactly the same.

@rwgk
rwgk requested a review from henryiii as a code ownerMay 1, 2021 00:06
@rwgk

rwgk commented May 4, 2021

Copy link
Copy Markdown
CollaboratorAuthor

Merging to cut down on number of PRs to manage. This change is still awaiting review under #2873.

@rwgk
rwgk merged commit bc3cd8a into pybind:smart_holderMay 4, 2021
@rwgk
rwgk deleted the sh_pragma_warning_block_h branch May 4, 2021 05:08
@github-actionsgithub-actionsBot added the needs changelog Possibly needs a changelog entry label May 4, 2021
@henryiiihenryiii removed the needs changelog Possibly needs a changelog entry label Jul 12, 2021
@rwgkrwgk mentioned this pull request Feb 10, 2023
Sign up for freeto 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

@rwgk@henryiii