Refactor `Error`s to strings that aren't used as `error`
Follow up to !399 (comment 455909360)
@jaime started a discussion !399 (comment 455909360):
If we are using them as error messages rather than errors, then we should probably change them to just string constants or even inline them in the log line itself. There's no point in defining an error that is not going to be returned or used as value, WDYT?
e.g. define the messages as strings
const ( createArtifactRequestErrMsg = "Failed to create the artifact request" artifactRequestErrMsg = "Failed to request the artifact" )
Edited by Tom Quirk