Skip to content

[636][CPyCppyy] Add pythonizations for string_view #19628

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

Merged
merged 1 commit into from
Aug 18, 2025

Conversation

dpiparo
Copy link
Member

@dpiparo dpiparo commented Aug 13, 2025

BP of #19620

This also makes sure that the `string_view` converts to Python strings
correctly.

Closes root-project#19476.

(cherry picked from commit d36655b)
@dpiparo dpiparo self-assigned this Aug 13, 2025
@dpiparo dpiparo requested a review from vepadulano as a code owner August 13, 2025 09:19
Copy link

github-actions bot commented Aug 13, 2025

Test Results

    19 files      19 suites   2d 23h 5m 33s ⏱️
 2 746 tests  2 745 ✅ 0 💤 1 ❌
50 443 runs  50 442 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit 10c7755.

♻️ This comment has been updated with latest results.

@guitargeek guitargeek added the clean build Ask CI to do non-incremental build on PR label Aug 15, 2025
@guitargeek guitargeek closed this Aug 15, 2025
@guitargeek guitargeek reopened this Aug 15, 2025
@guitargeek
Copy link
Contributor

Trying clean build to see if the TensorFlow problem persists.

@dpiparo dpiparo closed this Aug 15, 2025
@dpiparo dpiparo reopened this Aug 15, 2025
@dpiparo dpiparo merged commit 9162b95 into root-project:v6-36-00-patches Aug 18, 2025
94 of 147 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clean build Ask CI to do non-incremental build on PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants