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.

Sign up for GitHub

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

Jump to bottom

test LAGraph with valgrind #173

Open

DrTimothyAldenDavis opened this issue Mar 22, 2023 · 2 comments

Member

DrTimothyAldenDavis commented Mar 22, 2023

We need to test LAGraph with valgrind, in the CI. I just found a few memory leaks I had overlooked. I've fixed them but there could be more.

Member Author

DrTimothyAldenDavis commented Mar 22, 2023

See for example https://iramykytyn.dev/how-to-integrate-valgrind-into-github-actions

Contributor

szarnyasg commented Mar 31, 2023 •

edited

Loading

I'm glad to help with writing the CI configuration for this. Which part would you specifically like to run in the CI using valgrind? All tests?

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