Skip to content

Commit 34960ef

Browse files
committed
switching from pkg/errors to errors
1 parent e9de5f5 commit 34960ef

File tree

8 files changed

+17
-21
lines changed

8 files changed

+17
-21
lines changed

cli/cmd/config.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ package cmd
33
import (
44
"context"
55
"database/sql"
6+
"errors"
67
"fmt"
78
mssql "github.com/denisenkom/go-mssqldb"
89
"github.com/denisenkom/go-mssqldb/azuread"
@@ -14,7 +15,6 @@ import (
1415

1516
_ "github.com/denisenkom/go-mssqldb/azuread"
1617
"github.com/denisenkom/go-mssqldb/msdsn"
17-
"github.com/pkg/errors"
1818
"github.com/sirupsen/logrus"
1919
"gopkg.in/yaml.v3"
2020
)
@@ -48,7 +48,7 @@ func OpenSocks5Sql(dsn string) (*sql.DB, error) {
4848
if socksProxyAddress != "" {
4949
dialer, err := proxy.SOCKS5("tcp", socksProxyAddress, nil, nil)
5050
if err != nil {
51-
return nil, errors.Wrap(err, fmt.Sprintf("Could not connect with SOCKS5 to %s", socksProxyAddress))
51+
return nil, fmt.Errorf("could not connect with SOCKS5 to %s because of: %w", socksProxyAddress, err)
5252
}
5353
connector.Dialer = dialer.(proxy.ContextDialer)
5454
}
@@ -70,7 +70,7 @@ func LoadConfig() (Config, error) {
7070

7171
configFilename := path.Join(directory, "sqlcode.yaml")
7272
if _, err := os.Stat(configFilename); os.IsNotExist(err) {
73-
return Config{}, errors.New("No sqlcode.yaml found in current directory")
73+
return Config{}, errors.New("no sqlcode.yaml found in current directory")
7474
}
7575

7676
yamlFile, err := ioutil.ReadFile(configFilename)

cli/cmd/constants.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
package cmd
22

33
import (
4+
"errors"
45
"fmt"
56

6-
"github.com/pkg/errors"
77
"github.com/spf13/cobra"
88
)
99

@@ -14,7 +14,7 @@ var (
1414
RunE: func(cmd *cobra.Command, args []string) error {
1515
if len(args) != 0 {
1616
_ = cmd.Help()
17-
return errors.New("Too many arguments")
17+
return errors.New("too many arguments")
1818
}
1919
d, err := dep(true)
2020
if err != nil {

cli/cmd/dep.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
package cmd
22

33
import (
4+
"errors"
45
"fmt"
56
"os"
67

7-
"github.com/pkg/errors"
88
"github.com/spf13/cobra"
99
"github.com/vippsas/sqlcode"
1010
)
@@ -27,7 +27,7 @@ var (
2727
RunE: func(cmd *cobra.Command, args []string) error {
2828
if len(args) != 0 {
2929
_ = cmd.Help()
30-
return errors.New("Too many arguments")
30+
return errors.New("too many arguments")
3131
}
3232
d, err := dep(true)
3333
if err != nil {

deployable.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,14 +3,14 @@ package sqlcode
33
import (
44
"context"
55
"database/sql"
6+
"errors"
67
"fmt"
78
"io/fs"
89
"strconv"
910
"strings"
1011
"time"
1112

1213
mssql "github.com/denisenkom/go-mssqldb"
13-
"github.com/pkg/errors"
1414
"github.com/vippsas/sqlcode/sqlparser"
1515
)
1616

@@ -220,7 +220,7 @@ func (d Deployable) IntConst(s string) (int, error) {
220220
return strconv.Atoi(declare.Literal.RawValue)
221221
}
222222
}
223-
return 0, errors.New("No `declare `" + s + "` found")
223+
return 0, fmt.Errorf("no `declare %s found`", s)
224224
}
225225

226226
func (d Deployable) MustIntConst(s string) int {

go.mod

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ require (
66
github.com/alecthomas/repr v0.4.0
77
github.com/denisenkom/go-mssqldb v0.12.3
88
github.com/gofrs/uuid v4.4.0+incompatible
9-
github.com/pkg/errors v0.9.1
109
github.com/sirupsen/logrus v1.9.3
1110
github.com/smasher164/xid v0.1.2
1211
github.com/spf13/cobra v1.9.1

go.sum

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,6 @@ github.com/modocache/gover v0.0.0-20171022184752-b58185e213c5/go.mod h1:caMODM3P
2626
github.com/pkg/browser v0.0.0-20180916011732-0a3d74bf9ce4/go.mod h1:4OwLy04Bl9Ef3GJJCoec+30X3LQs/0/m4HFRt/2LUSA=
2727
github.com/pkg/browser v0.0.0-20240102092130-5ac0b6a4141c h1:+mdjkGKdHQG3305AYmdv1U2eRNDiU2ErMBj1gwrq8eQ=
2828
github.com/pkg/browser v0.0.0-20240102092130-5ac0b6a4141c/go.mod h1:7rwL4CYBLnjLxUqIJNnCWiEdr3bn6IUYi15bNlnbCCU=
29-
github.com/pkg/errors v0.9.1 h1:FEBLx1zS214owpjy7qsBeixbURkuhQAwrK5UwLGTwt4=
30-
github.com/pkg/errors v0.9.1/go.mod h1:bwawxfHBFNV+L2hUp1rHADufV3IMtnDRdf1r5NINEl0=
3129
github.com/pmezard/go-difflib v1.0.0 h1:4DBwDE0NGyQoBHbLQYPwSUPoCMWR5BEzIk/f1lZbAQM=
3230
github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
3331
github.com/russross/blackfriday/v2 v2.1.0/go.mod h1:+Rmxgy9KzJVeS9/2gXHxylqXiyQDYRxCVz55jmeOWTM=

preprocess.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@ package sqlcode
33
import (
44
"crypto/sha256"
55
"encoding/hex"
6+
"errors"
67
"fmt"
7-
"github.com/pkg/errors"
88
"github.com/vippsas/sqlcode/sqlparser"
99
"regexp"
1010
"strings"

sqltest/querydump.go

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ import (
99
"time"
1010

1111
"github.com/alecthomas/repr"
12-
"github.com/pkg/errors"
1312
)
1413

1514
type MapRow map[string]interface{}
@@ -24,7 +23,7 @@ func runQuery(dbi interface{}, qry string, args ...interface{}) *sql.Rows {
2423
case CtxQuerier:
2524
rows, err := q.QueryContext(context.Background(), qry, args...)
2625
if err != nil {
27-
panic(errors.WithStack(err))
26+
panic(fmt.Sprintf("runQuery, query: %s \n\n arguments:%+v \n\n error: %s", qry, args, err))
2827
}
2928
return rows
3029
default:
@@ -38,11 +37,11 @@ func RowIteratorToSlice(rows *sql.Rows) (columns []string, result Rows) {
3837

3938
columns, err := rows.Columns()
4039
if err != nil {
41-
panic(errors.WithStack(err))
40+
panic(fmt.Sprintf("RowIteratorToSlice: while getting columns: %s", err)
4241
}
4342
types, err := rows.ColumnTypes()
4443
if err != nil || len(types) != len(columns) {
45-
panic(errors.WithStack(err))
44+
panic(fmt.Sprintf("RowIteratorToSlice: while getting columns types: %s", err)
4645
}
4746
n := len(columns)
4847
rowValues := make([]interface{}, n, n)
@@ -53,7 +52,7 @@ func RowIteratorToSlice(rows *sql.Rows) (columns []string, result Rows) {
5352
for rows.Next() {
5453
err = rows.Scan(pointers...)
5554
if err != nil {
56-
panic(errors.WithStack(err))
55+
panic(fmt.Sprintf("RowIteratorToSlice: while scanning: %s", err)
5756
}
5857

5958
var row Row
@@ -130,21 +129,21 @@ func Query(dbi CtxQuerier, qry string, args ...interface{}) Rows {
130129

131130
func QueryInt(dbi CtxQuerier, qry string, args ...interface{}) (result int) {
132131
if err := dbi.QueryRowContext(context.Background(), qry, args...).Scan(&result); err != nil {
133-
panic(errors.WithStack(err))
132+
panic(fmt.Sprintf("QueryInt, query: %s\n\n arguments:%+v\n\n error: %s", qry, args, err))
134133
}
135134
return
136135
}
137136

138137
func QueryString(dbi CtxQuerier, qry string, args ...interface{}) (result string) {
139138
if err := dbi.QueryRowContext(context.Background(), qry, args...).Scan(&result); err != nil {
140-
panic(errors.WithStack(err))
139+
panic(fmt.Sprintf("QueryString, query: %s\n\n arguments:%+v\n\n error: %s", qry, args, err))
141140
}
142141
return
143142
}
144143

145144
func QueryTime(dbi CtxQuerier, qry string, args ...interface{}) (result time.Time) {
146145
if err := dbi.QueryRowContext(context.Background(), qry, args...).Scan(&result); err != nil {
147-
panic(errors.WithStack(err))
146+
panic(fmt.Sprintf("QueryTime, query: %s\n\n arguments:%+v\n\n error: %s", qry, args, err))
148147
}
149148
return
150149
}

0 commit comments

Comments
 (0)