Merge pull request #41 from matsuri-tech/feat/lint-and-format

feat: todoに入れてたlintを適用

Author: iori-ikeda

.github/workflows/lint.yml

@@ -22,4 +22,4 @@ jobs:
         uses: golangci/golangci-lint-action@v4
         with:
           version: latest
-          args: --config=.golangci.yml -v --timeout 10m
+          args: --config=.golangci.yml -v --timeout 1m

.golangci.yml

@@ -1,23 +1,13 @@
 linters:
-  # disable-all: true
   enable:
     - gofumpt
     - durationcheck
     - forcetypeassert
-    # TODO: そのうち入れたいlinterのオプション
-    # - gocognit
-    # - goconst
-    # - gocritic
-    #   - disabled-checks:
-    #     - captLocal
-    # - gochecknoglobals
-    # - errorlint
-    # - exhaustive
-    # - funlen
+    - gocognit
+    - goconst
+    - gocritic
+    - gochecknoglobals
+    - errorlint
+    - exhaustive
 issues:
   max-same-issues: 0
-  exclude-rules:
-    # Exclude some linters from running on tests files.
-    - path: _test\.go
-      linters:
-        - forcetypeassert

endpoints.go

@@ -228,17 +228,18 @@ func (e *endpoints) generateOpenApiSchema(config OpenApiGeneratorConfig) (openap
 			item = paths[path]
 		}
 
-		if api.Method == http.MethodGet {
+		switch api.Method {
+		case http.MethodGet:
 			item.Get = &operation
-		} else if api.Method == http.MethodPost {
+		case http.MethodPost:
 			item.Post = &operation
 			operation.RequestBody = &requestBodyAny
-		} else if api.Method == http.MethodPut {
+		case http.MethodPut:
 			item.Put = &operation
 			operation.RequestBody = &requestBodyAny
-		} else if api.Method == http.MethodDelete {
+		case http.MethodDelete:
 			item.Delete = &operation
-		} else if api.Method == http.MethodPatch {
+		case http.MethodPatch:
 			item.Patch = &operation
 			operation.RequestBody = &requestBodyAny
 		}