diff options
| author | silverwind <me@silverwind.io> | 2024-06-11 20:47:45 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-06-11 18:47:45 +0000 |
| commit | fc2d75f86d77b022ece848acf2581c14ef21d43b (patch) | |
| tree | c95effed84f093643d8408b7cd32123a1114962d /services/pull/merge.go | |
| parent | 4bf848a06bfa069e5f381235193924f2b35f2d9d (diff) | |
| download | gitea-fc2d75f86d77b022ece848acf2581c14ef21d43b.tar.gz gitea-fc2d75f86d77b022ece848acf2581c14ef21d43b.zip | |
Enable `unparam` linter (#31277)
Enable [unparam](https://github.com/mvdan/unparam) linter.
Often I could not tell the intention why param is unused, so I put
`//nolint` for those cases like webhook request creation functions never
using `ctx`.
---------
Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
Co-authored-by: delvh <dev.lh@web.de>
Diffstat (limited to 'services/pull/merge.go')
| -rw-r--r-- | services/pull/merge.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/services/pull/merge.go b/services/pull/merge.go index 6b5e9ea330..9ef3fb2e05 100644 --- a/services/pull/merge.go +++ b/services/pull/merge.go @@ -246,7 +246,7 @@ func Merge(ctx context.Context, pr *issues_model.PullRequest, doer *user_model.U } // doMergeAndPush performs the merge operation without changing any pull information in database and pushes it up to the base repository -func doMergeAndPush(ctx context.Context, pr *issues_model.PullRequest, doer *user_model.User, mergeStyle repo_model.MergeStyle, expectedHeadCommitID, message string, pushTrigger repo_module.PushTrigger) (string, error) { +func doMergeAndPush(ctx context.Context, pr *issues_model.PullRequest, doer *user_model.User, mergeStyle repo_model.MergeStyle, expectedHeadCommitID, message string, pushTrigger repo_module.PushTrigger) (string, error) { //nolint:unparam // Clone base repo. mergeCtx, cancel, err := createTemporaryRepoForMerge(ctx, pr, doer, expectedHeadCommitID) if err != nil { |