add error check to JSON unmarshal, ci fails if linting fails

[davidbloss]

Issues

Changelog

Removed .golangci.yml file - this was disabling some error checking
Check for errors on json.Unmarshall()

• NewTestRequest() vets test data and verifies it's working with valid JSON. Checking for JSON errors is now a safe move.

task lint now returns a non-zero return code so we can enforce this in CI

• List your changes here
• Make a changie entry

Tophatting

task test - tests pass

[codecov]

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (b81d606) 76.80% compared to head (ca8bf15) 76.75%.

Files	Patch %	Lines
json.go	0.00%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #306      +/-   ##
==========================================
- Coverage   76.80%   76.75%   -0.06%     
==========================================
  Files          50       50              
  Lines        3307     3308       +1     
==========================================
- Hits         2540     2539       -1     
- Misses        564      565       +1     
- Partials      203      204       +1     

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