Catch2 update 2.13.4

[zoq]

Catch2 update 2.13.4 which will fix #266.

[rcurtin]

I'm not sure why the memory tests are failing (looks like they ran fine); the static code analysis issues can be ignored; and the Windows build issues appear to be #267 (so this PR wouldn't solve them anyway). So, looks good otherwise! 😄

[zoq]

I'm not sure why the memory tests are failing (looks like they ran fine); the static code analysis issues can be ignored; and the Windows build issues appear to be #267 (so this PR wouldn't solve them anyway). So, looks good otherwise!

The memory job run into the build timeout which is set to 360 minutes, usually we just run the memory check on changed tests, but since an update of the catch2 header will trigger all tests to be run we run into the timeout. But agreed, save to ignore in this case.

[rcurtin]

Ahh, ok, thanks---I must have just missed the 'job terminated' line in the log. 👍

[mlpack-bot]

Second approval provided automatically after 24 hours. 👍