diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index c84ed6f..d5659ca 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -31,12 +31,6 @@ jobs: with: go-version: ${{ matrix.go-version }} cache: true - - - name: Download dependencies - run: go mod download - - - name: Verify dependencies - run: go mod verify - name: Build binary run: go build -v -o bin/git-get ./cmd/ diff --git a/.golangci.yml b/.golangci.yml index 5d0d076..59c2706 100644 --- a/.golangci.yml +++ b/.golangci.yml @@ -19,7 +19,7 @@ linters: - godox # TODO: enable it and handle all the remaning TODOs - mnd # Impractical. We deal with numbers like file permissions here, it's much clearer to see them explicitly. - noinlineerr # Impractical. Inline error handling is a common and idiomatic practice - - testpackage # TODO: renable it and refactor tests into separate packages + - testpackage # Impractical for CLI apps. This linter is more valuable for libraries to ensure a clean public API. - unparam # Impractical, it flags functions that are designed to be general-purpose, but happen to only be used with specific values currently - wsl # We use wsl_v5 instead - wrapcheck # Adds too much bloat, many of the errors are contextual enough and don't need wrapping