Skip to content

Turn cppyy on llvm 20 jobs #582

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mcbarton
Copy link
Collaborator

Description

Please include a summary of changes, motivation and context for this PR.

This PR will turn on cppyy for the llvm 20 jobs. This will require an identical PR in cppyy to create any needed Valgrind suppression files.

Fixes # (issue)

Type of change

Please tick all options which are relevant.

  • Bug fix
  • New feature
  • Requires documentation updates

Testing

Please describe the test(s) that you added and ran to verify your changes.

Checklist

  • I have read the contribution guide recently

Copy link

codecov bot commented May 19, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.45%. Comparing base (03135b6) to head (5a8f4ab).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #582   +/-   ##
=======================================
  Coverage   76.45%   76.45%           
=======================================
  Files           9        9           
  Lines        3648     3648           
=======================================
  Hits         2789     2789           
  Misses        859      859           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mcbarton
Copy link
Collaborator Author

@Vipul-Cariappa @aaronj0 I am going to open the cppyy counterpart to this PR sometime next week. I will likely to be able to generate the needed suppression files, but lack the knowledge to interpret them once generated, so will need your help there to see if anything in there is problematic.

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.

1 participant