djpbessems/ContainerImage.Pinniped
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use the "Err*" idiomatic naming for error variables/consts.

Browse Source 
Signed-off-by: Matt Moyer <moyerm@vmware.com>
This commit is contained in:
Matt Moyer 2020-07-28 08:37:27 -05:00
parent 27cd82065b
commit a5dbc324f6
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
cmd/placeholder-name/main.go
View File

@ -28,7 +28,7 @@ func main() {
type envGetter func(string) (string, bool) type envGetter func(string) (string, bool)
type tokenExchanger func(token, caBundle, apiEndpoint string) (*clientauthentication.ExecCredential, error) type tokenExchanger func(token, caBundle, apiEndpoint string) (*clientauthentication.ExecCredential, error)
const EnvVarNotSetError = constable.Error("failed to login: environment variable not set") const ErrMissingEnvVar = constable.Error("failed to login: environment variable not set")
func run(envGetter envGetter, tokenExchanger tokenExchanger, outputWriter io.Writer) error { func run(envGetter envGetter, tokenExchanger tokenExchanger, outputWriter io.Writer) error {
token, varExists := envGetter("PLACEHOLDER_NAME_TOKEN") token, varExists := envGetter("PLACEHOLDER_NAME_TOKEN")
@ -60,5 +60,5 @@ func run(envGetter envGetter, tokenExchanger tokenExchanger, outputWriter io.Wri
} }
func envVarNotSetError(varName string) error { func envVarNotSetError(varName string) error {
return fmt.Errorf("%w: %s", EnvVarNotSetError, varName) return fmt.Errorf("%w: %s", ErrMissingEnvVar, varName)
} }