Skip to content

Commit 88f5df5

Browse files
author
Mattia Bertorello
committed
Fixed all the problem reported by ineffassign ( ineffectual assignment)
1 parent 75de366 commit 88f5df5

File tree

3 files changed

+5
-4
lines changed

3 files changed

+5
-4
lines changed

.golangci.yml

-1
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@ linters:
2525
- goconst
2626
- gocyclo
2727
- govet
28-
- ineffassign
2928
- interfacer
3029
- lll
3130
- maligned

commands/commands_test.go

+3-1
Original file line numberDiff line numberDiff line change
@@ -206,8 +206,10 @@ func TestUserLibs(t *testing.T) {
206206
func TestLibDownloadAndInstall(t *testing.T) {
207207
defer makeTempDataDir(t)()
208208
defer makeTempSketchbookDir(t)()
209+
var d []byte
210+
var exitCode int
209211

210-
exitCode, d := executeWithArgs(t, "core", "update-index")
212+
exitCode, _ = executeWithArgs(t, "core", "update-index")
211213
require.Zero(t, exitCode, "exit code")
212214

213215
// Download inexistent

commands/compile/compile.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ func run(cmd *cobra.Command, args []string) {
106106

107107
// Check for ctags tool
108108
loadBuiltinCtagsMetadata(pm)
109-
ctags, err := getBuiltinCtagsTool(pm)
109+
ctags, _ := getBuiltinCtagsTool(pm)
110110
if !ctags.IsInstalled() {
111111
formatter.Print("Downloading and installing missing tool: " + ctags.String())
112112
core.DownloadToolRelease(pm, ctags)
@@ -116,7 +116,7 @@ func run(cmd *cobra.Command, args []string) {
116116
formatter.PrintError(err, "Could not load hardware packages.")
117117
os.Exit(commands.ErrCoreConfig)
118118
}
119-
ctags, err = getBuiltinCtagsTool(pm)
119+
ctags, _ = getBuiltinCtagsTool(pm)
120120
if !ctags.IsInstalled() {
121121
formatter.PrintErrorMessage("Missing ctags tool.")
122122
os.Exit(commands.ErrCoreConfig)

0 commit comments

Comments
 (0)