Replace `const std::string&` with `std::string_view` where it makes sense
As mentioned in !963 (merged), now that we're supporting C++17, it would be nice to replace our usages of const std::string&
with std::string_view
where it makes sense to do so, see some hints/ideas here.