This pull request refactors the code to replace instances of os.Setenv with t.Setenv in test files. The t.Setenv method, introduced in Go 1.17, is specifically designed for managing environment variables in tests, ensuring they are properly cleaned up after each test case. currently both are in use within the provider and this aligns them with the correct methold.
attempted to enable nilerr to enforce a nolint directive & a comment explaining why the error is being ignored, however linter does not seem to respect nolint director so left comments and disabled linter
also tidy up the golangci config file
This is a (please select all that apply):
[ ] Bug Fix
[ ] New Feature (ie adding a service, resource, or data source)
[x] Enhancement
[ ] Breaking Change
Related Issue(s)
Fixes #0000
[!NOTE]
If this PR changes meaningfully during the course of review please update the title and description as required.
This pull request refactors the code to replace instances of os.Setenv with t.Setenv in test files. The t.Setenv method, introduced in Go 1.17, is specifically designed for managing environment variables in tests, ensuring they are properly cleaned up after each test case. currently both are in use within the provider and this aligns them with the correct methold.
attempted to enable nilerr to enforce a nolint directive & a comment explaining why the error is being ignored, however linter does not seem to respect nolint director so left comments and disabled linter
also tidy up the golangci config file
This is a (please select all that apply):
Related Issue(s)
Fixes #0000