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

[v632] Backport of recent fixes and improvements #15486

Merged
merged 7 commits into from
May 13, 2024

Conversation

guitargeek
Copy link
Contributor

Backport of recent fixes and improvements. The fix for the cppyy regression is important in particular.

Copy link

Test Results

     9 files       9 suites   1d 20h 34m 11s ⏱️
 2 630 tests  2 630 ✅ 0 💤 0 ❌
22 143 runs  22 143 ✅ 0 💤 0 ❌

Results for commit 7ca32f5.

Copy link
Member

@hahnjo hahnjo left a comment

Choose a reason for hiding this comment

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

Please see #15483 (review); IMHO commits should only be backported once they landed in master...

@guitargeek
Copy link
Contributor Author

guitargeek commented May 13, 2024

Sure! I just opened the PR already because it's easy to do in one go, and then I don't forget it later :) If the master commits get some changes because of review requests, it's easy to update the backport PR and merge once the master PRs are merged. Also, I can get 6.32 test coverage already.

@hahnjo hahnjo requested review from hahnjo and removed request for hahnjo May 13, 2024 15:38
@guitargeek guitargeek merged commit 3c60d2d into root-project:v6-32-00-patches May 13, 2024
11 of 13 checks passed
@guitargeek guitargeek deleted the backports_v632 branch May 13, 2024 17:10
@guitargeek
Copy link
Contributor Author

The commits that are not merged in master were removed from this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants