add unparam as a presubmit check

[julieqiu]

unparam is a common linter for Go projects. We should consider adding it as a presubmit check for PRs to catch things like googleapis/gapic-generator-go#1407

See googleapis/gapic-generator-go#1408 for a related issue.

Example issues in this repository:

auth  ❯ unparam ./...                                                                   
credentials/impersonate/user.go:33:123: user - result 1 (error) is always nil
internal/transport/cert/cmd/test_signer.go:59:95: (*EnterpriseCertSigner).CertificateChain - result 0 (error) is always nil

bigquery ❯ unparam ./...                                                               
iterator.go:369:22: fetchCachedPage - ctx is unused
model.go:232:54: bqToModelMetadata - result 1 (error) is always nil
routine.go:240:87: bqToRemoteFunctionOptions - result 1 (error) is always nil
routine.go:258:70: (*RemoteFunctionOptions).toBQ - result 1 (error) is always nil
storage/managedwriter/adapt/protoconversion.go:414:158: tableFieldSchemaToFieldDescriptorProto - result 1 (error) is always nil
storage/managedwriter/managed_stream.go:197:60: (*ManagedStream).appendWithRetry - opts is unused
storage/managedwriter/managed_stream.go:358:57: (*ManagedStream).processRetry - srcConn is unused
storage/managedwriter/retry.go:36:46: retryPredicate - result aggressiveBackoff is always false

[quartzmo]

Great suggestion, thanks!