AgreeableDeerGames / GameBackbone

2D game framework
MIT License
11 stars 4 forks source link

Run Dr. Memory over tests in windows CI #246

Open lavinrp opened 4 years ago

lavinrp commented 4 years ago

Is your feature request related to a problem? Please describe. Currently we dont have any memory checking in our CI pipeline since we can't run on linux. This can result in some hard to track bugs.

Describe the solution you'd like Run Dr. Memory over the unit tests in the windows pipeline. Dr. Memory is a tool similar to valgrind memcheck.

Describe alternatives you've considered We could run various sanitizers over our code (ASan, UBSan, etc...) These solutions are not mutually exclusive.

Dr. Memory should be easier to set up though since it isn't an ABI break and doesn't require recompiling all of our dependencies.

Additional context GitHub Actions runners have the Chocolatey package manager. It should be possible to install Dr. Memory from here.;