Skip to content

Commit

Permalink
refactor(cli): remove unnecessary log statement (#6983)
Browse files Browse the repository at this point in the history
This no longer provides any useful information now that gazelle is
patched to walk subdirectories instead of us doing it manually here.

---

### Changes are visible to end-users: no

### Test plan

- Covered by existing test cases

GitOrigin-RevId: 8cdd9e60d027f509099508b0e0b0a085afc0772d
  • Loading branch information
jbedard committed Oct 23, 2024
1 parent 62685ba commit 51d669e
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions gazelle/common/walk.go
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
package gazelle

import (
"path/filepath"

BazelLog "aspect.build/cli/pkg/logger"
"github.com/bazelbuild/bazel-gazelle/language"
)
Expand All @@ -24,10 +22,7 @@ func GazelleWalkDir(args language.GenerateArgs, walkFunc GazelleWalkFunc) error
continue
}

BazelLog.Tracef("GazelleWalkDir RegularFile: %s", f)

walkErr := walkFunc(f)
if walkErr != nil && walkErr != filepath.SkipDir {
if walkErr := walkFunc(f); walkErr != nil {
return walkErr
}
}
Expand Down

0 comments on commit 51d669e

Please sign in to comment.