re-enabled valgrind in CI (#4516)

.github/workflows/asan.yml

@@ -52,7 +52,7 @@ jobs:
       - name: Run tests
         run: ./cmake.output/bin/testrunner
 
-      # TODO: re-enable - was being killed because of increased memory usage
+      # TODO: this is currently way too slow (~60 minutes) to enable it
       # TODO: only fail the step on sanitizer issues
       - name: Self check
         if: false

.github/workflows/valgrind.yml

@@ -2,8 +2,7 @@
 # Environment reference https://help.github.com/en/actions/reference/virtual-environments-for-github-hosted-runners
 name: valgrind
 
-# on: [push, pull_request]
-on: workflow_dispatch
+on: [push, pull_request]
 
 permissions:
   contents: read