-
Notifications
You must be signed in to change notification settings - Fork 427
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
fix: linting errors #1432
fix: linting errors #1432
Conversation
Integration tests failure for dab7f39648623a82e1158040e574a957b7cfa350 |
Integration tests failure for 98a1dddb751b4cbe6b48a0d1aca2b9cfdb904f81 |
Integration tests success for 443d56761798c705dd54f61e7ea93cdca621df5a |
Integration tests failure for a240a6645f14e592b05f45afd3b7a0be8ac314a9 |
pkg/validation/validation.go
Outdated
} | ||
n.WriteString(fmt.Sprintf(`."%v"`, objectName)) | ||
return n.String() | ||
} else { // schemaName != "" |
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.
🚫 [golangci] reported by reviewdog 🐶
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (revive)
Integration tests failure for 0cd6039f6eec1b2770f2abb369f1b5a0b6d28a87 |
Integration tests failure for b7f96307d068865b9e0c05d01dcb56474512deb9 |
Integration tests failure for 5b357741361445d7bc17e9993dd02b4c8619e993 |
Integration tests failure for bde100b120fdc2cd58b5235f58a48bcce0eb9144 |
Integration tests failure for b0be914983f39702b4877b6ccb3fe70c5d02a0f2 |
Integration tests failure for 351a42a1cf02972646b1c4b83781681a5130ff1d |
Integration tests success for d33e245b8d8186aaa3b97ff3e97c1373ce291c49 |
Test Plan
References