Skip to content

[CPyCppyy] Avoid compiler warnings with Python 3.15 - #22545

Merged
guitargeek merged 1 commit into
root-project:masterfrom
guitargeek:py315
Jun 10, 2026
Merged

[CPyCppyy] Avoid compiler warnings with Python 3.15#22545
guitargeek merged 1 commit into
root-project:masterfrom
guitargeek:py315

Conversation

@guitargeek

Copy link
Copy Markdown
Contributor

This avoids compiler warnings about missing field initializers because of a new data member in Python 3.15.

Also, refactor the code a bit, so that next time a new member is added, we just have to change the code in one place.

This avoids compiler warnings about missing field initializers because
of a new data member in Python 3.15.

Also, refactor the code a bit, so that next time a new member is added,
we just have to change the code in one place.
@guitargeek
guitargeek requested a review from aaronj0 June 9, 2026 13:38
@guitargeek guitargeek self-assigned this Jun 9, 2026
@guitargeek
guitargeek requested a review from vepadulano as a code owner June 9, 2026 13:38
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

Test Results

    22 files      22 suites   3d 12h 25m 11s ⏱️
 3 864 tests  3 856 ✅   0 💤 8 ❌
76 324 runs  76 106 ✅ 209 💤 9 ❌

For more details on these failures, see this check.

Results for commit a9c80cc.

♻️ This comment has been updated with latest results.

@vepadulano vepadulano left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but let's wait for the CI to be green.

@guitargeek

Copy link
Copy Markdown
Contributor Author

Failures unrelated to Python.

@guitargeek
guitargeek merged commit 917096a into root-project:master Jun 10, 2026
52 of 58 checks passed
@guitargeek
guitargeek deleted the py315 branch June 10, 2026 15:02
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