Modernize (#277)

* fix: format code with gofumpt

Signed-off-by: Federico Guerinoni <[email protected]>

* ci: bump GH actions

Signed-off-by: Federico Guerinoni <[email protected]>

* chore: drop support go 1.17

* chore: update golangci-linter

Signed-off-by: Federico Guerinoni <[email protected]>

---------

Signed-off-by: Federico Guerinoni <[email protected]>

Author: guerinoni

.github/workflows/ci.yml

@@ -17,12 +17,12 @@ jobs:
       max-parallel: 3
       fail-fast: false
       matrix:
-        goVer: ['1.17', '1.18', '1.19', '1.20', '1.21', '1.22']
+        goVer: ['1.18', '1.19', '1.20', '1.21', '1.22']
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
 
       - name: Set up Go ${{ matrix.goVer }}
-        uses: actions/setup-go@v3
+        uses: actions/setup-go@v5
         with:
           go-version: ${{ matrix.goVer }}
 

.github/workflows/initiate_release.yml

@@ -12,7 +12,7 @@ jobs:
     name: 🚀 Create release PR
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
         with:
           fetch-depth: 0 # gives the changelog generator access to all previous commits
 

.github/workflows/lint.yml

@@ -12,15 +12,15 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
         with:
           fetch-depth: 0
 
       - name: Commit message linter
         uses: wagoid/commitlint-github-action@v4
 
       - name: Setup Go
-        uses: actions/setup-go@v3
+        uses: actions/setup-go@v5
         with:
           go-version: "1.18"
 

.github/workflows/release.yml

@@ -12,9 +12,7 @@ jobs:
     if: github.event.pull_request.merged && startsWith(github.head_ref, 'release-')
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
-        with:
-          fetch-depth: 0
+      - uses: actions/checkout@v4
 
       - uses: actions/github-script@v6
         with:

.github/workflows/reviewdog.yml

@@ -11,16 +11,16 @@ jobs:
     name: 🐶 Reviewdog
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
 
       - uses: reviewdog/action-setup@v1
         with:
           reviewdog_version: latest
 
       - name: Setup Go
-        uses: actions/setup-go@v3
+        uses: actions/setup-go@v5
         with:
-          go-version: '1.17'
+          go-version: '1.18'
 
       - name: Install golangci-lint
         run: make install-golangci

.golangci.yml

@@ -38,24 +38,19 @@ run:
 linters:
   enable-all: true
   disable:
-    - contextcheck
-    - exhaustivestruct
+    - depguard
+    - tagalign
+    - err113
     - exhaustruct
-    - funlen
     - gochecknoglobals
     - godox
-    - goerr113
-    - golint
     - gomnd
-    - interfacer
     - lll
-    - maligned
+    - mnd
     - nlreturn
     - nonamedreturns
     - paralleltest
-    - scopelint
     - tagliatelle
-    - thelper
     - varnamelen
     - wrapcheck
     - wsl
@@ -74,3 +69,8 @@ issues:
       linters:
         - gosec
         - testpackage # makes you use a separate _test package
+        - gochecknoinits
+        - depguard
+        - testableexamples
+        - contextcheck
+        - funlen

Makefile

@@ -1,4 +1,4 @@
-GOLANGCI_VERSION = 1.46.2
+GOLANGCI_VERSION = 1.58.2
 GOLANGCI = vendor/golangci-lint/$(GOLANGCI_VERSION)/golangci-lint
 
 install-golangci:

app.go

@@ -100,7 +100,7 @@ func NewAppSettings() *AppSettings {
 
 type AsyncModerationCallback struct {
 	Mode      string `json:"mode"`
-	ServerUrl string `json:"server_url"`
+	ServerURL string `json:"server_url"`
 }
 
 type AsyncModerationConfiguration struct {

app_test.go

@@ -36,7 +36,7 @@ func TestClient_CheckAsyncModeConfig(t *testing.T) {
 			AsyncModerationConfiguration{
 				Callback: &AsyncModerationCallback{
 					Mode:      "CALLBACK_MODE_REST",
-					ServerUrl: "https://example.com/gosdk",
+					ServerURL: "https://example.com/gosdk",
 				},
 				Timeout: 10000,
 			},

async_tasks.go

@@ -33,7 +33,7 @@ type TaskResponse struct {
 // GetTask returns the status of a task that has been ran asynchronously.
 func (c *Client) GetTask(ctx context.Context, id string) (*TaskResponse, error) {
 	if id == "" {
-		return nil, fmt.Errorf("id should not be empty")
+		return nil, errors.New("id should not be empty")
 	}
 
 	p := path.Join("tasks", url.PathEscape(id))
@@ -53,7 +53,7 @@ type AsyncTaskResponse struct {
 // It returns an AsyncTaskResponse object which contains the task ID, the status of the task can be check with client.GetTask method.
 func (c *Client) DeleteChannels(ctx context.Context, cids []string, hardDelete bool) (*AsyncTaskResponse, error) {
 	if len(cids) == 0 {
-		return nil, fmt.Errorf("cids parameter should not be empty")
+		return nil, errors.New("cids parameter should not be empty")
 	}
 
 	data := struct {
@@ -91,7 +91,7 @@ type DeleteUserOptions struct {
 // It returns an AsyncTaskResponse object which contains the task ID, the status of the task can be check with client.GetTask method.
 func (c *Client) DeleteUsers(ctx context.Context, userIDs []string, options DeleteUserOptions) (*AsyncTaskResponse, error) {
 	if len(userIDs) == 0 {
-		return nil, fmt.Errorf("userIDs parameter should not be empty")
+		return nil, errors.New("userIDs parameter should not be empty")
 	}
 
 	data := struct {