Skip to content

Valgrind CI #565

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 9 commits into from
Jun 1, 2024
Merged

Valgrind CI #565

merged 9 commits into from
Jun 1, 2024

Conversation

jacobwilliams
Copy link
Owner

@jacobwilliams jacobwilliams commented Jun 1, 2024

Fixes #563

Copy link

codecov bot commented Jun 1, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.67%. Comparing base (46dcb62) to head (7afba2b).
Report is 5 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #565      +/-   ##
==========================================
- Coverage   89.48%   88.67%   -0.82%     
==========================================
  Files           7        7              
  Lines        5356     5358       +2     
==========================================
- Hits         4793     4751      -42     
- Misses        563      607      +44     

@jacobwilliams jacobwilliams merged commit 23a597d into master Jun 1, 2024
12 checks passed
@jacobwilliams jacobwilliams deleted the 563-valgrind-CI branch June 1, 2024 21:37
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add valgrind to CI
1 participant