ci(lint): refactor code for clarity and linting compliance #289

Merged
appleboy merged 1 commits from refs/pull/289/head into main 2023-07-13 01:10:57 +00:00
appleboy commented 2023-07-13 01:08:40 +00:00 (Migrated from gitea.com)
  • Removed deadcode, structcheck, and varcheck linters from .golangci.yml
  • Fixed a typo in a comment in daemon.go
  • Renamed defaultActionsUrl to defaultActionsURL in exec.go
  • Removed unnecessary else clause in exec.go and runner.go
  • Simplified variable initialization in exec.go
  • Changed function name from getHttpClient to getHTTPClient in http.go
  • Removed unnecessary else clause in labels_test.go

Signed-off-by: Bo-Yi Wu appleboy.tw@gmail.com

- Removed `deadcode`, `structcheck`, and `varcheck` linters from `.golangci.yml` - Fixed a typo in a comment in `daemon.go` - Renamed `defaultActionsUrl` to `defaultActionsURL` in `exec.go` - Removed unnecessary else clause in `exec.go` and `runner.go` - Simplified variable initialization in `exec.go` - Changed function name from `getHttpClient` to `getHTTPClient` in `http.go` - Removed unnecessary else clause in `labels_test.go` Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>
lunny (Migrated from gitea.com) reviewed 2023-07-13 01:09:11 +00:00
wolfogre (Migrated from gitea.com) reviewed 2023-07-13 01:09:12 +00:00
lunny (Migrated from gitea.com) approved these changes 2023-07-13 01:10:07 +00:00
Sign in to join this conversation.