chore: enable unused-parameter from revive

[mmorel-35]

Thank you for contributing to Velero!

Please add a summary of your change

enable and fixes unused-parameter rule from revive

Does your change fix a particular issue?

Fixes #(issue)

Please indicate you've done the following:

• Accepted the DCO. Commits without the DCO will delay acceptance.
• Created a changelog file (make new-changelog) or comment /kind changelog-not-required on this PR.
• Updated the corresponding documentation in site/content/docs/main.

[mmorel-35]

/kind changelog-not-required

[codecov]

Codecov Report

Attention: Patch coverage is 37.03704% with 102 lines in your changes missing coverage. Please review.

Project coverage is 59.34%. Comparing base (5b1738a) to head (327ec28).

Files with missing lines	Patch %	Lines
pkg/plugin/framework/volume_snapshotter_server.go	0.00%	7 Missing ⚠️
pkg/repository/provider/restic.go	0.00%	7 Missing ⚠️
pkg/plugin/framework/object_store_server.go	0.00%	6 Missing ⚠️
pkg/util/kube/predicate.go	0.00%	6 Missing ⚠️
pkg/plugin/framework/action_resolver.go	0.00%	5 Missing ⚠️
pkg/datapath/micro_service_watcher.go	0.00%	4 Missing ⚠️
pkg/cmd/cli/snapshotlocation/set.go	0.00%	3 Missing ⚠️
pkg/controller/gc_controller.go	0.00%	3 Missing ⚠️
...reitemaction/v2/restartable_restore_item_action.go	0.00%	3 Missing ⚠️
pkg/cmd/cli/install/install.go	0.00%	2 Missing ⚠️
... and 45 more

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #8625      +/-   ##
==========================================
+ Coverage   59.33%   59.34%   +0.01%     
==========================================
  Files         370      370              
  Lines       39932    39932              
==========================================
+ Hits        23692    23697       +5     
+ Misses      14744    14739       -5     
  Partials     1496     1496              

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

[reasonerjt]

Thanks for the PR!

I believe this rule has some negative effects.
When we want to use a parameter in the func which is not used before, the name of the parameter, though it's not used, is helpful to help the developer understand how it should be used.

for example, when you change "func f(BackupID string)" to "func f(_ string)", it makes the future change a little more difficult.

[reasonerjt]

I'm blocking this PR until the maintainers reach agreement regarding whether the rule should be enforced.