chore(cicd): updating the cicd to include the testify lint #126
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Describe your changes
This pull request includes updates to the
.golangci.yaml
configuration file and multiple test files to enhance code quality and consistency. The primary changes involve adding a new linter and updating test assertions to use therequire
package for better error handling.Configuration Updates:
.golangci.yaml
: Addedtestifylint
to the list of enabled linters and configured it to enable all rules. [1] [2]Test Enhancements:
inserter/sql_test.go
: Updated test assertions to uses.Require()
instead ofs.NoError()
ands.Len()
for more consistent and immediate test failures. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14]loader_test.go
: Similar updates were made to uses.Require()
instead ofs.NoError()
ands.Error()
across various test cases to ensure immediate and clear test failures. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16] [17] [18] [19] [20] [21] [22] [23]