The StatusCode enum is currently defined via c2hs. This means that the Haskell enumeration includes a constructor StatusCodeDoNotUse since we convert directly from the C enumeration. As the name suggests, we probably should not have a constructor for StatusCodeDoNotUse. The googleapis gRPC reflection protos excludes a do_not_use enum from their status code enum. If possible, I suggest that we redefine the StatusCode enumeration so that a StatusCodeDoNotUse constructor is no longer included. However, this would require that we redefine StatusCode as a typical Haskell enum that exactly matches the grpc_status_code enumeration 1-to-1 other than the do_not_use case. Moreover, we would need to ensure that making this change does not have a meaningful impact at the gRPC-Haskell to grpc-core boundry.
I can make this change unless someone raises a good reason as to why we should include StatusCodeDoNotUse.
The
StatusCodeenum is currently defined via c2hs. This means that the Haskell enumeration includes a constructorStatusCodeDoNotUsesince we convert directly from the C enumeration. As the name suggests, we probably should not have a constructor forStatusCodeDoNotUse. ThegoogleapisgRPC reflection protos excludes a do_not_use enum from their status code enum. If possible, I suggest that we redefine theStatusCodeenumeration so that aStatusCodeDoNotUseconstructor is no longer included. However, this would require that we redefineStatusCodeas a typical Haskell enum that exactly matches thegrpc_status_codeenumeration 1-to-1 other than thedo_not_usecase. Moreover, we would need to ensure that making this change does not have a meaningful impact at thegRPC-Haskelltogrpc-coreboundry.I can make this change unless someone raises a good reason as to why we should include
StatusCodeDoNotUse.