A while ago clang-tidy started failing in CI [1] and the intermediate solution was to disable the task
so it would not block merges of incoming pull requests. Running clang_tidy locally it looks like we've
been building up some tech debt with respect to cleanly passing the clang_tidy check.
Filing this issue as a hook for an upcoming PR to address that.

[1] #546