-
Notifications
You must be signed in to change notification settings - Fork 41
Conversation
We are removing the catchError that we previously added
💔 Tests FailedExpand to view the summary
Build stats
Test stats 🧪
Test errorsExpand to view the tests failures
Steps errorsExpand to view the steps failures
Log outputExpand to view the last 100 lines of log output
|
We are removing the catchError that we previously added
We are removing the catchError that we previously added
What does this PR do?
We are removing any reference to golintci, plus the catchError that we previously added
Why is it important?
Because of project layout, a monorepo, the golint many times reports false positives with Go code. The usage of a (powerful) IDE, such as VSCode, where we leverage the compiler and tooling to detect lint errors, allows to remove this with certain degree of coonfidence.
make notice
in the proper directory)Author's Checklist