Remove redundant steps from CI workflow

- go mod verify is not needed when go.sum is checked in. go build and go
test will automatically verify checksums
- go mod download is handled by the setup-go action already

.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/

.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