Remove NOLINT pragmas from ErrorCode macro.

[davidlion]

Request

In error_handling/ErrorCode.hpp there is the following macro that we use NOLINT to avoid clang-tidy issues.

// NOLINTBEGIN(bugprone-macro-parentheses, cppcoreguidelines-macro-usage)
#define CLP_ERROR_HANDLING_MARK_AS_ERROR_CODE_ENUM(T) \
    template <> \
    struct std::is_error_code_enum<clp::error_handling::ErrorCode<T>> : std::true_type { \
        static_assert(std::is_enum_v<T>); \
    };
// NOLINTEND(bugprone-macro-parentheses, cppcoreguidelines-macro-usage)

After skimming the clang-tidy documentation for the two issues [1, 2] it seems possible to satisfy these checks, but more investigation is needed.

refs:

1. https://clang.llvm.org/extra/clang-tidy/checks/bugprone/macro-parentheses.html
2. https://clang.llvm.org/extra/clang-tidy/checks/cppcoreguidelines/macro-usage.html

Possible implementation

For cppcoreguidelines-macro-usage:

1. Investigate whether there is another construct to use
2. Use the AllowedRegexp option to allow the macro

For bugprone-macro-parentheses:

1. Add parentheses as necessary to satisfy