Skip to content

Commit

Permalink
bug: lint error
Browse files Browse the repository at this point in the history
  • Loading branch information
katallaxie authored Nov 17, 2024
1 parent d2699c7 commit 26b65ff
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 1 deletion.
1 change: 0 additions & 1 deletion pkg/sources/reconciler/azservicebussource/adapter.go
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,6 @@ func MakeAppEnv(o *v1alpha1.AzureServiceBusSource) []corev1.EnvVar {
}

var appEnvs []corev1.EnvVar
//TODO should this be here?
if sasAuth := o.Spec.Auth.SASToken; sasAuth != nil {
appEnvs = common.MaybeAppendValueFromEnvVar(appEnvs, common.EnvServiceBusKeyName, sasAuth.KeyName)
appEnvs = common.MaybeAppendValueFromEnvVar(appEnvs, common.EnvServiceBusKeyValue, sasAuth.KeyValue)
Expand Down
3 changes: 3 additions & 0 deletions pkg/sources/reconciler/azservicebussource/subscription.go
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,8 @@ const crudTimeout = time.Second * 15
// Required permissions:
// - Microsoft.ServiceBus/namespaces/topics/subscriptions/read
// - Microsoft.ServiceBus/namespaces/topics/subscriptions/write
//
// nolint:gocyclo
func EnsureSubscription(ctx context.Context, cli servicebustopics.SubscriptionsClient) error {
if skip.Skip(ctx) {
return nil
Expand Down Expand Up @@ -174,6 +176,7 @@ func recursErrMsg(errMsg string, err error) string {
errMsg += ": "
}

// nolint:errorlint
switch tErr := err.(type) {
case autorest.DetailedError:
return recursErrMsg(errMsg+tErr.Message, tErr.Original)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,8 @@ const crudTimeout = time.Second * 15
// Required permissions:
// - Microsoft.ServiceBus/namespaces/topics/subscriptions/read
// - Microsoft.ServiceBus/namespaces/topics/subscriptions/write
//
// nolint:gocyclo
func EnsureSubscription(ctx context.Context, cli servicebustopics.SubscriptionsClient) error {
if skip.Skip(ctx) {
return nil
Expand Down Expand Up @@ -174,6 +176,7 @@ func recursErrMsg(errMsg string, err error) string {
errMsg += ": "
}

// nolint:errorlint
switch tErr := err.(type) {
case autorest.DetailedError:
return recursErrMsg(errMsg+tErr.Message, tErr.Original)
Expand Down

0 comments on commit 26b65ff

Please sign in to comment.