refactor: enable linter rule ST1008 for 'error should be last return value'

[mwbrooks]

Summary

This pull request enables the linter staticcheck rule ST1008: 'A function’s error value should be its last return value'.

✅ func HelloWorld (string, error)
❌ func HelloWorld (error, string)

Requirements

• I've read and understood the Contributing Guidelines and have done my best effort to follow them.
• I've read and agree to the Code of Conduct.

[codecov]

Codecov Report

Attention: Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.

Project coverage is 63.16%. Comparing base (f21cbc2) to head (79dd545).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
cmd/root.go	0.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #76      +/-   ##
==========================================
- Coverage   63.16%   63.16%   -0.01%     
==========================================
  Files         210      210              
  Lines       22187    22187              
==========================================
- Hits        14015    14014       -1     
- Misses       7083     7086       +3     
+ Partials     1089     1087       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[zimeg]

@mwbrooks I like this rule so much! 🤖 ✨

Before merging we might want to change one more instance of erroneous error returns:

slack-cli/internal/hooks/sdk_config.go

Line 49 in eca699f

func (s *SDKCLIConfig) Exists() (error, bool) {

I'm not sure how the linter missed this, but it's been bugging me for a while 😉

[mwbrooks]

Good catch @zimeg! 🎉 It's strange that the linter didn't find that one.

Commit c5a8bbd updates func (s *SDKCLIConfig) Exists() (bool, error) {