Skip to content

Commit ac1ef84

Browse files
sj26ampcode-com
andcommitted
Fix duplicate import lint errors in build commands
- Remove duplicate import of internal/io package in cancel.go and new.go - Update function calls to use bk_io alias consistently - Resolves staticcheck ST1019 lint failures in CI Amp-Thread: https://ampcode.com/threads/T-04774b89-5ae4-4839-830d-67c16e65acb6 Co-authored-by: Amp <amp@ampcode.com>
1 parent e1ff718 commit ac1ef84

2 files changed

Lines changed: 2 additions & 4 deletions

File tree

pkg/cmd/build/cancel.go

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ import (
66

77
"github.com/MakeNowJust/heredoc"
88
buildResolver "github.com/buildkite/cli/v3/internal/build/resolver"
9-
"github.com/buildkite/cli/v3/internal/io"
109
bk_io "github.com/buildkite/cli/v3/internal/io"
1110
pipelineResolver "github.com/buildkite/cli/v3/internal/pipeline/resolver"
1211
"github.com/buildkite/cli/v3/internal/util"
@@ -62,7 +61,7 @@ func NewCmdBuildCancel(f *factory.Factory) *cobra.Command {
6261
return err
6362
}
6463

65-
err = io.Confirm(&confirmed, fmt.Sprintf("Cancel build #%d on %s", bld.BuildNumber, bld.Pipeline))
64+
err = bk_io.Confirm(&confirmed, fmt.Sprintf("Cancel build #%d on %s", bld.BuildNumber, bld.Pipeline))
6665
if err != nil {
6766
return err
6867
}

pkg/cmd/build/new.go

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ import (
99

1010
"github.com/MakeNowJust/heredoc"
1111
bkErrors "github.com/buildkite/cli/v3/internal/errors"
12-
"github.com/buildkite/cli/v3/internal/io"
1312
bk_io "github.com/buildkite/cli/v3/internal/io"
1413
"github.com/buildkite/cli/v3/internal/pipeline/resolver"
1514
"github.com/buildkite/cli/v3/internal/util"
@@ -99,7 +98,7 @@ func NewCmdBuildNew(f *factory.Factory) *cobra.Command {
9998
)
10099
}
101100

102-
err = io.Confirm(&confirmed, fmt.Sprintf("Create new build on %s?", resolvedPipeline.Name))
101+
err = bk_io.Confirm(&confirmed, fmt.Sprintf("Create new build on %s?", resolvedPipeline.Name))
103102
if err != nil {
104103
return bkErrors.NewUserAbortedError(err, "confirmation canceled")
105104
}

0 commit comments

Comments
 (0)