[golangci] reported by reviewdog 🐶 func `(*zookeeperFetcher).disconnect` is unused (unused) Raw Output: modules/zookeeper/fetcher.go:46:28: func `(*zookeeperFetcher).disconnect` is unused (unused) func (c *zookeeperFetcher) disconnect() {} ^

[golangci] reported by reviewdog 🐶 field `doCheck` is unused (unused) Raw Output: modules/windows/windows.go:84:3: field `doCheck` is unused (unused) doCheck bool ^

[golangci] reported by reviewdog 🐶 func `(*runningJobs).has` is unused (unused) Raw Output: agent/jobmgr/cache.go:158:23: func `(*runningJobs).has` is unused (unused) func (c *runningJobs) has(fullName string) bool { ^

[golangci] reported by reviewdog 🐶 func `(*retryingTasks).has` is unused (unused) Raw Output: agent/jobmgr/cache.go:181:25: func `(*retryingTasks).has` is unused (unused) func (c *retryingTasks) has(cfg confgroup.Config) bool { ^

[golangci] reported by reviewdog 🐶 S1023: redundant `return` statement (gosimple) Raw Output: agent/jobmgr/manager.go:235:2: S1023: redundant `return` statement (gosimple) return ^

[golangci] reported by reviewdog 🐶 SA9003: empty branch (staticcheck) Raw Output: agent/jobmgr/manager.go:174:3: SA9003: empty branch (staticcheck) if m.FileStatusStore.Contains(ecfg.cfg, "") { ^