Skip to content

Commit 72c455d

Browse files
fix: appese govet linter
non-constant format string in call. We were calling some format function, but we were always passing a "static" string.
1 parent 5bd18b2 commit 72c455d

File tree

4 files changed

+6
-7
lines changed

4 files changed

+6
-7
lines changed

commands/instances.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -340,7 +340,7 @@ func (s *arduinoCoreServerImpl) Init(req *rpc.InitRequest, stream rpc.ArduinoCor
340340
logrus.WithField("index", indexFile).Info("Loading libraries index file")
341341
li, err := librariesindex.LoadIndex(indexFile)
342342
if err != nil {
343-
s := status.Newf(codes.FailedPrecondition, i18n.Tr("Loading index file: %v", err))
343+
s := status.New(codes.FailedPrecondition, i18n.Tr("Loading index file: %v", err))
344344
responseError(s)
345345
li = librariesindex.EmptyIndex
346346
}

internal/arduino/builder/recipe.go

+2-3
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@
1616
package builder
1717

1818
import (
19-
"fmt"
2019
"sort"
2120
"strings"
2221

@@ -27,7 +26,7 @@ import (
2726

2827
// RunRecipe fixdoc
2928
func (b *Builder) RunRecipe(prefix, suffix string, skipIfOnlyUpdatingCompilationDatabase bool) error {
30-
logrus.Debugf(fmt.Sprintf("Looking for recipes like %s", prefix+"*"+suffix))
29+
logrus.Debugf("Looking for recipes like %s", prefix+"*"+suffix)
3130

3231
// TODO is it necessary to use Clone?
3332
buildProperties := b.buildProperties.Clone()
@@ -36,7 +35,7 @@ func (b *Builder) RunRecipe(prefix, suffix string, skipIfOnlyUpdatingCompilation
3635
// TODO is it necessary to use Clone?
3736
properties := buildProperties.Clone()
3837
for _, recipe := range recipes {
39-
logrus.Debugf(fmt.Sprintf("Running recipe: %s", recipe))
38+
logrus.Debugf("Running recipe: %s", recipe)
4039

4140
command, err := b.prepareCommandForRecipe(properties, recipe, false)
4241
if err != nil {

internal/arduino/libraries/librariesmanager/librariesmanager.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -210,7 +210,7 @@ func (lm *LibrariesManager) loadLibrariesFromDir(librariesDir *LibrariesDir) []*
210210
return statuses
211211
}
212212
if err != nil {
213-
s := status.Newf(codes.FailedPrecondition, i18n.Tr("reading dir %[1]s: %[2]s", librariesDir.Path, err))
213+
s := status.New(codes.FailedPrecondition, i18n.Tr("reading dir %[1]s: %[2]s", librariesDir.Path, err))
214214
return append(statuses, s)
215215
}
216216
d.FilterDirs()
@@ -221,7 +221,7 @@ func (lm *LibrariesManager) loadLibrariesFromDir(librariesDir *LibrariesDir) []*
221221
for _, libDir := range libDirs {
222222
library, err := libraries.Load(libDir, librariesDir.Location)
223223
if err != nil {
224-
s := status.Newf(codes.Internal, i18n.Tr("loading library from %[1]s: %[2]s", libDir, err))
224+
s := status.New(codes.Internal, i18n.Tr("loading library from %[1]s: %[2]s", libDir, err))
225225
statuses = append(statuses, s)
226226
continue
227227
}

internal/arduino/libraries/librariesresolver/cpp.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ func (resolver *Cpp) ResolveFor(header, architecture string) *libraries.Library
123123
logrus.
124124
WithField("lib", lib.Name).
125125
WithField("prio", fmt.Sprintf("%03X", libPriority)).
126-
Infof(msg)
126+
Info(msg)
127127
}
128128
if found == nil {
129129
return nil

0 commit comments

Comments
 (0)