feat(body): added additional attribute '--body' to dsv secret create & dsv home c…

[pacificcode]

added additional attribute '--body' to dsv secret create & dsv home.

Lint future work: AB#561862 AB#561861

[codecov]

Codecov Report

Attention: Patch coverage is 48.00000% with 13 lines in your changes are missing coverage. Please review.

Project coverage is 51.09%. Comparing base (a2521eb) to head (f41db62).
Report is 94 commits behind head on main.

Files	Patch %	Lines
commands/secret.go	40.00%	10 Missing and 2 partials ⚠️
format/writers.go	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##             main     #134       +/-   ##
===========================================
+ Coverage   32.61%   51.09%   +18.48%     
===========================================
  Files          80       87        +7     
  Lines       10855    11964     +1109     
===========================================
+ Hits         3540     6113     +2573     
+ Misses       7027     5551     -1476     
- Partials      288      300       +12     

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

[sheldonhull]

@pacificcode try running trunk check locally and should get 3 items flagged. Maybe existing code or not. If you need to add an exception you can document why that's a legit exception to bypass the check using the //nolint:<linter> // documented reason why no lint shouldn't apply. Try to resolve if straight-forward, low effort, otherwise document why.

[andrii-zakurenyi]

LGTM 👍

Then only minor thing is that I think it is a good idea to use any instead of interface{}. If you update that then I'll re-approve.

[sheldonhull]

For a failed checksum with Aqua in actions try Aqua update-checksum -prune and see if that file failed to get updated. It's a verification step so probably l missed being updated.

Don't forget to run changie new for chabgekog entry as well.

[sheldonhull]

I see the test action failed. That's the first time I've ever seen that message and has to do with the signature for Aqua tool (a safety net for verifying the binary). I'll look at this and post an update today after checking into it.

[sheldonhull]

pending batch commit changes and then this can be merged.