ClangTidy warnings for `cppcoreguidelines-pro-type-reinterpret-cast`

[jviotti]

/Users/jviotti/Projects/core/src/core/yaml/yaml.cc:22:11: error: do not use reinterpret_cast [cppcoreguidelines-pro-type-reinterpret-cast,-warnings-as-errors]
   22 |           reinterpret_cast<char *>(node->data.scalar.value),
      |           ^

/Users/jviotti/Projects/core/src/core/yaml/yaml.cc:71:25: error: do not use reinterpret_cast [cppcoreguidelines-pro-type-reinterpret-cast,-warnings-as-errors]
   71 |           result.assign(reinterpret_cast<char *>(key_node->data.scalar.value),
      |                         ^

/Users/jviotti/Projects/core/src/core/yaml/yaml.cc:125:16: error: do not use reinterpret_cast [cppcoreguidelines-pro-type-reinterpret-cast,-warnings-as-errors]
  125 |       &parser, reinterpret_cast<const unsigned char *>(input.c_str()),
      |                ^

[jviotti]

I'm not sure we can solve this one. I get the rule, but a low level reinterpret_cast is often needed when dealing with C code, like we are doing here. We might just need to add some NOLINT directives