Skip to content

Commit

Permalink
Merge pull request #17 from Venafi/fix-pipeline-lint
Browse files Browse the repository at this point in the history
Fixes lint issue for printing error and removes track of idea folder
  • Loading branch information
luispresuelVenafi authored Sep 11, 2024
2 parents 56809e7 + f8d34cf commit f216f13
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,4 @@
/.terraform.lock.hcl
/pkg/
/terraform.tfstate
.idea
3 changes: 2 additions & 1 deletion internal/provider/credential_resource.go
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ package provider

import (
"context"
"errors"
"fmt"
"strconv"
"strings"
Expand Down Expand Up @@ -299,7 +300,7 @@ func getValuesMap(ctx context.Context, values string) (map[string]string, error)
if !found {
msg := fmt.Sprintf("no separator found on value: %s", item)
tflog.Info(ctx, msg)
return nil, fmt.Errorf(msg)
return nil, errors.New(msg)
}
tflog.Debug(ctx, fmt.Sprintf("credential field found: %s = %s", key, value))
dict[key] = value
Expand Down

0 comments on commit f216f13

Please sign in to comment.