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

feat: lint and format

Author: iori-ikeda

.github/workflows/lint.yml

@@ -0,0 +1,25 @@
+name: golang-ci-lint
+on:
+  push:
+    branches:
+      # デフォルトブランチに合わせて変更
+      - master
+  pull_request:
+
+permissions:
+  contents: read
+jobs:
+  golangci:
+    name: lint
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version-file: "./go.mod"
+          cache: true
+      - name: golangci-lint
+        uses: golangci/golangci-lint-action@v4
+        with:
+          version: latest
+          args: --config=.golangci.yml -v --timeout 10m

.golangci.yml

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

endpoints.go

@@ -4,12 +4,13 @@ import (
 	"bytes"
 	"encoding/json"
 	"fmt"
-	"github.com/invopop/jsonschema"
 	"io"
 	"net/http"
 	"os"
 	"strings"
 
+	"github.com/invopop/jsonschema"
+
 	"github.com/getkin/kin-openapi/openapi3"
 	"gopkg.in/yaml.v3"
 

endpoints_test.go

@@ -3,9 +3,10 @@ package endpoints
 import (
 	"bytes"
 	"encoding/json"
+	"testing"
+
 	"github.com/labstack/echo/v4"
 	"github.com/stretchr/testify/assert"
-	"testing"
 )
 
 type SampleModel struct {

wrapper.go

@@ -1,9 +1,10 @@
 package endpoints
 
 import (
-	"github.com/labstack/echo/v4"
 	"net/http"
 	"os"
+
+	"github.com/labstack/echo/v4"
 )
 
 // EchoWrapper に対応するメソッドが存在しないEchoの機能を使いたい場合に限り、
@@ -77,7 +78,7 @@ func (w *EchoWrapper) Generate(filename string) error {
 }
 
 func (w *EchoWrapper) GenerateOpenApiJson(filename string, config OpenApiGeneratorConfig) error {
-	file, err := os.OpenFile(filename, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, 0644)
+	file, err := os.OpenFile(filename, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, 0o644)
 	if err != nil {
 		return err
 	}
@@ -89,7 +90,7 @@ func (w *EchoWrapper) GenerateOpenApiJson(filename string, config OpenApiGenerat
 }
 
 func (w *EchoWrapper) GenerateOpenApi(filename string, config OpenApiGeneratorConfig) error {
-	file, err := os.OpenFile(filename, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, 0644)
+	file, err := os.OpenFile(filename, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, 0o644)
 	if err != nil {
 		return err
 	}