-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
🌱 make golangci-lint config consistent across repos #255
🌱 make golangci-lint config consistent across repos #255
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #255 +/- ##
=======================================
Coverage 48.84% 48.84%
=======================================
Files 8 8
Lines 434 434
=======================================
Hits 212 212
Misses 201 201
Partials 21 21 ☔ View full report in Codecov by Sentry. |
@@ -35,11 +35,11 @@ $(GINKGO): $(BINGO_DIR)/ginkgo.mod | |||
@echo "(re)installing $(GOBIN)/ginkgo-v2.12.0" | |||
@cd $(BINGO_DIR) && GOWORK=off $(GO) build -mod=mod -modfile=ginkgo.mod -o=$(GOBIN)/ginkgo-v2.12.0 "github.com/onsi/ginkgo/v2/ginkgo" | |||
|
|||
GOLANGCI_LINT := $(GOBIN)/golangci-lint-v1.53.3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do I really want these version numbers to be variables?
# If you are making a change, please make it in ALL | ||
# of the above repositories! | ||
# | ||
# TODO: Find a way to have a shared golangci config. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
vendor and symlink?
cmd/manager/main.go
Outdated
//+kubebuilder:scaffold:imports | ||
"github.com/operator-framework/catalogd/api/core/v1alpha1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems bad. I think I'll just delete this line.
cmd/manager/main.go
Outdated
// Import all Kubernetes client auth plugins (e.g. Azure, GCP, OIDC, etc.) | ||
// to ensure that exec-entrypoint and run can make use of them. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm going to delete this line as well. IIRC, it is scaffolded with Kubebuilder/Operator SDK. It is meant to go with the _ "k8s.io/client-go/plugin/pkg/client/auth"
import, but it seems to always get jumbled with import formaters run.
Signed-off-by: Joe Lanford <joe.lanford@gmail.com>
00c819f
to
623470d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
No description provided.