Skip to content
This repository has been archived by the owner on Jan 8, 2024. It is now read-only.

Commit

Permalink
Remove else error check
Browse files Browse the repository at this point in the history
Registries aren't required to have access info funcs, so remove this
check and try the build anyway
  • Loading branch information
briancain committed Oct 20, 2021
1 parent d62b5a6 commit 5e743dc
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions internal/core/app_build.go
Original file line number Diff line number Diff line change
Expand Up @@ -177,13 +177,10 @@ func (op *buildOperation) Do(ctx context.Context, log hclog.Logger, app *App, _
return nil, err
}
} else {
if ra != nil && ra.AccessInfoFunc() == nil {
if ok && ra != nil && ra.AccessInfoFunc() == nil {
return nil, status.Error(codes.Internal, "The plugin requested does not "+
"define an AccessInfoFunc() in its Registry plugin. This is an internal "+
"error and should be reported to the author of the plugin.")
} else {
return nil, status.Error(codes.Internal, "The given build operation value "+
"is not a component.RegistryAccess type.")
}
}
}
Expand Down

0 comments on commit 5e743dc

Please sign in to comment.