Skip to content
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

chore: Add golangci-lint and fix most issues #505

Merged
merged 1 commit into from
Jun 7, 2024
Merged

chore: Add golangci-lint and fix most issues #505

merged 1 commit into from
Jun 7, 2024

Commits on Jun 5, 2024

  1. chore: Add golangci-lint and fix most issues

    The new `make lint` target runs golangci-lint if installed locally. It
    is not (yet) required for a passing build.
    
    Most issues reported by golangci-lint are resolved. Some interesting
    ones:
    
    * Missing error checking is added for exporting to a properties file and
      to Terraform variables.
    * Unused functions are removed: stringInSlice, S3KMSStore.readObject.
    * Unused variables are removed: validKeyFormat.
    * Write operations used to set up store tests now have their errors
      checked. If a setup write fails, the test immediately fails.
    
    Unresolved issues:
    
    * Use of EndpointResolverWithOptions
    * Missing error checks for flushing, syncing, and closing a file
      created by the export command
    bhavanki committed Jun 5, 2024
    Configuration menu
    Copy the full SHA
    a693d26 View commit details
    Browse the repository at this point in the history