Stricter Linter Rules

[mattdot]

Draft only because this just enables the rules. We can use this branch to also fix the rules before merging.

[eduardodfmex]

This pull request includes significant changes to the codebase, primarily focusing on renaming the ApiClient to Client, enabling and configuring linters, and making minor improvements to code readability and consistency.

Renaming ApiClient to Client:

• internal/api/api_client.go: Renamed ApiClient to Client and updated all related method signatures to reflect this change. [1] [2] [3] [4] [5]
• internal/api/api_client_test.go: Updated test cases to use Client instead of ApiClient.
• internal/api/auth.go: Updated references from ApiClient to Client within the auth.go file. [1] [2] [3]
• internal/api/lifecycle.go: Changed method signature to use Client instead of ApiClient.
• internal/api/request.go: Updated method signatures and references to use Client instead of ApiClient. [1] [2] [3] [4]

Enabling and Configuring Linters:

• .golangci.yml: Enabled several linters including revive, staticcheck, errcheck, and configured specific rules for revive linter.

Code Readability and Consistency Improvements:

• internal/helpers/config.go: Simplified conditional return statements. [1] [2]
• internal/helpers/config_test.go: Added comments to clarify the purpose of not running tests in parallel. [1] [2] [3]
• internal/constants/constants.go: Added new constants HTTPS and API_VERSION_PARAM.

Minor Refactoring:

• internal/customtypes/uuid_type.go: Updated method signatures to include ctx parameter for consistency.
• internal/customtypes/uuid_value.go: Updated method signatures to include ctx parameter for consistency. [1] [2] [3]