WIP

Author: unknwon

.editorconfig

@@ -0,0 +1,9 @@
+# Docs: http://editorconfig.org
+
+root = true
+
+[*]
+charset = utf-8
+end_of_line = lf
+insert_final_newline = true
+trim_trailing_whitespace = true

.github/workflows/go.yml

@@ -1,8 +1,18 @@
 name: Go
 on:
   push:
-    branches: [master]
+    branches: [ main ]
+    paths:
+      - '**.go'
+      - 'go.mod'
+      - '.golangci.yml'
+      - '.github/workflows/go.yml'
   pull_request:
+    paths:
+      - '**.go'
+      - 'go.mod'
+      - '.golangci.yml'
+      - '.github/workflows/go.yml'
 env:
   GOPROXY: "https://proxy.golang.org"
 
@@ -11,35 +21,36 @@ jobs:
     name: Lint
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
+      - name: Install Go
+        uses: actions/setup-go@v3
+        with:
+          go-version: 1.20.x
+      - name: Checkout code
+        uses: actions/checkout@v3
       - name: Run golangci-lint
-        uses: actions-contrib/golangci-lint@v1
+        uses: golangci/golangci-lint-action@v3
+        with:
+          version: latest
+          args: --timeout=30m
 
   test:
     name: Test
     strategy:
       matrix:
-        go-version: [1.14.x, 1.15.x]
-        platform: [ubuntu-latest, macos-latest, windows-latest]
+        go-version: [ 1.19.x, 1.20.x ]
+        platform: [ ubuntu-latest, macos-latest, windows-latest ]
     runs-on: ${{ matrix.platform }}
     steps:
       - name: Install Go
-        uses: actions/setup-go@v1
+        uses: actions/setup-go@v3
         with:
           go-version: ${{ matrix.go-version }}
       - name: Checkout code
-        uses: actions/checkout@v2
-      - name: Run unit tests
+        uses: actions/checkout@v3
+      - name: Run tests with coverage
         run: go test -v -race -coverprofile=coverage -covermode=atomic ./...
       - name: Upload coverage report to Codecov
-        uses: codecov/codecov-action@v1.0.6
+        uses: codecov/codecov-action@v1.5.0
         with:
           file: ./coverage
           flags: unittests
-      - name: Cache downloaded modules
-        uses: actions/cache@v1
-        with:
-          path: ~/go/pkg/mod
-          key: ${{ runner.os }}-go-${{ hashFiles('**/go.sum') }}
-          restore-keys: |
-            ${{ runner.os }}-go-

.github/workflows/lsif.yml

@@ -1,17 +1,28 @@
 name: LSIF
-on: [push]
+on:
+  push:
+    paths:
+      - '**.go'
+      - 'go.mod'
+      - '.github/workflows/lsif.yml'
+env:
+  GOPROXY: "https://proxy.golang.org"
+
 jobs:
-  build:
+  lsif-go:
+    if: github.repository == 'go-clog/clog'
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v1
+      - uses: actions/checkout@v3
       - name: Generate LSIF data
         uses: sourcegraph/lsif-go-action@master
+      - name: Upload LSIF data to sourcegraph.com
+        continue-on-error: true
+        uses: docker://sourcegraph/src-cli:latest
         with:
-          verbose: 'true'
-      - name: Upload LSIF data
-        uses: sourcegraph/lsif-upload-action@master
+          args: lsif upload -github-token=${{ secrets.GITHUB_TOKEN }}
+      - name: Upload LSIF data to cs.unknwon.dev
         continue-on-error: true
+        uses: docker://sourcegraph/src-cli:latest
         with:
-          endpoint: https://sourcegraph.com
-          github_token: ${{ secrets.GITHUB_TOKEN }}
+          args: -endpoint=https://cs.unknwon.dev lsif upload -github-token=${{ secrets.GITHUB_TOKEN }}

.golangci.yml

@@ -0,0 +1,19 @@
+linters-settings:
+  nakedret:
+    max-func-lines: 0 # Disallow any unnamed return statement
+
+linters:
+  enable:
+    - unused
+    - errcheck
+    - gosimple
+    - govet
+    - ineffassign
+    - staticcheck
+    - typecheck
+    - nakedret
+    - gofmt
+    - rowserrcheck
+    - unconvert
+    - goimports
+    - unparam

README.md

@@ -1,4 +1,4 @@
-# Clog 
+# Clog
 
 [![GitHub Workflow Status](https://img.shields.io/github/workflow/status/go-clog/clog/Go?logo=github&style=for-the-badge)](https://github.com/go-clog/clog/actions?query=workflow%3AGo)
 [![codecov](https://img.shields.io/codecov/c/github/go-clog/clog/master?logo=codecov&style=for-the-badge)](https://codecov.io/gh/go-clog/clog)
@@ -13,10 +13,10 @@ This package supports multiple loggers across different levels of logging. It us
 
 ## Installation
 
-The minimum requirement of Go is **1.11**.
+The minimum requirement of Go is **1.19**.
 
 	go get unknwon.dev/clog/v2
-    
+
 Please apply `-u` flag to update in the future.
 
 ## Getting Started
@@ -51,7 +51,7 @@ The code inside `init` function is equivalent to the following:
 
 ```go
 func init() {
-	err := log.NewConsole(0, 
+	err := log.NewConsole(0,
         log.ConsoleConfig{
 		    Level: log.LevelTrace,
 	    },
@@ -66,7 +66,7 @@ Or expand further:
 
 ```go
 func init() {
-	err := log.NewConsoleWithName(log.DefaultConsoleName, 0, 
+	err := log.NewConsoleWithName(log.DefaultConsoleName, 0,
         log.ConsoleConfig{
 		    Level: log.LevelTrace,
 	    },
@@ -85,7 +85,7 @@ In production, you may want to make log less verbose and be asynchronous:
 
 ```go
 func init() {
-	// The buffer size mainly depends on number of logs could be produced at the same time, 
+	// The buffer size mainly depends on number of logs could be produced at the same time,
 	// 100 is a good default.
 	err := log.NewConsole(100,
         log.ConsoleConfig{
@@ -145,7 +145,7 @@ func main() {
 
 ### Caller Location
 
-When using `log.Error` and `log.Fatal` functions, the caller location is written along with logs. 
+When using `log.Error` and `log.Fatal` functions, the caller location is written along with logs.
 
 ```go
 func main() {
@@ -171,10 +171,10 @@ File logger is the single most powerful builtin logger, it has the ability to ro
 
 ```go
 func init() {
-	err := log.NewFile(100, 
+	err := log.NewFile(100,
         log.FileConfig{
             Level:              log.LevelInfo,
-            Filename:           "clog.log",  
+            Filename:           "clog.log",
             FileRotationConfig: log.FileRotationConfig {
                 Rotate: true,
                 Daily:  true,

discord.go

@@ -6,7 +6,6 @@ import (
 	"errors"
 	"fmt"
 	"io"
-	"io/ioutil"
 	"net/http"
 	"strings"
 	"time"
@@ -117,7 +116,7 @@ func (l *discordLogger) postMessage(r io.Reader) (int64, error) {
 
 		return rateLimitMsg.RetryAfter, nil
 	} else if resp.StatusCode/100 != 2 {
-		data, err := ioutil.ReadAll(resp.Body)
+		data, err := io.ReadAll(resp.Body)
 		if err != nil {
 			return -1, fmt.Errorf("read HTTP response body: %v", err)
 		}

discord_test.go

@@ -4,7 +4,7 @@ import (
 	"bytes"
 	"encoding/json"
 	"errors"
-	"io/ioutil"
+	"io"
 	"net/http"
 	"testing"
 
@@ -283,7 +283,7 @@ func Test_discordLogger_postMessage(t *testing.T) {
 
 				return &http.Response{
 					StatusCode: statusCode,
-					Body:       ioutil.NopCloser(bytes.NewBufferString(respBody)),
+					Body:       io.NopCloser(bytes.NewBufferString(respBody)),
 					Header:     make(http.Header),
 				}
 			}),

file.go

@@ -4,7 +4,6 @@ import (
 	"bytes"
 	"fmt"
 	"io"
-	"io/ioutil"
 	"log"
 	"os"
 	"path/filepath"
@@ -121,7 +120,7 @@ func (l *fileLogger) initRotation() error {
 
 	// If there is any content in the file, count the number of lines.
 	if l.rotationConfig.MaxLines > 0 && l.currentSize > 0 {
-		data, err := ioutil.ReadFile(l.filename)
+		data, err := os.ReadFile(l.filename)
 		if err != nil {
 			return fmt.Errorf("read file %q: %v", l.filename, err)
 		}

file_test.go

@@ -2,7 +2,6 @@ package clog
 
 import (
 	"errors"
-	"io/ioutil"
 	"os"
 	"path/filepath"
 	"runtime"
@@ -74,11 +73,11 @@ func Test_rotateFilename(t *testing.T) {
 
 	filename := rotateFilename("test/Test_rotateFilename.log", "2017-03-05")
 	assert.Equal(t, "test/Test_rotateFilename.log.2017-03-05", filename)
-	assert.Nil(t, ioutil.WriteFile(filename, []byte(""), os.ModePerm))
+	assert.Nil(t, os.WriteFile(filename, []byte(""), os.ModePerm))
 
 	filename = rotateFilename("test/Test_rotateFilename.log", "2017-03-05")
 	assert.Equal(t, "test/Test_rotateFilename.log.2017-03-05.001", filename)
-	assert.Nil(t, ioutil.WriteFile(filename, []byte(""), os.ModePerm))
+	assert.Nil(t, os.WriteFile(filename, []byte(""), os.ModePerm))
 
 	filename = rotateFilename("test/Test_rotateFilename.log", "2017-03-05")
 	assert.Equal(t, "test/Test_rotateFilename.log.2017-03-05.002", filename)

go.mod

@@ -1,10 +1,17 @@
 module unknwon.dev/clog/v2
 
-go 1.11
+go 1.19
 
 require (
 	github.com/fatih/color v1.7.0
+	github.com/stretchr/testify v1.4.0
+)
+
+require (
+	github.com/davecgh/go-spew v1.1.0 // indirect
 	github.com/mattn/go-colorable v0.1.4 // indirect
 	github.com/mattn/go-isatty v0.0.10 // indirect
-	github.com/stretchr/testify v1.4.0
+	github.com/pmezard/go-difflib v1.0.0 // indirect
+	golang.org/x/sys v0.0.0-20191008105621-543471e840be // indirect
+	gopkg.in/yaml.v2 v2.2.2 // indirect
 )

slack.go

@@ -6,7 +6,6 @@ import (
 	"errors"
 	"fmt"
 	"io"
-	"io/ioutil"
 	"net/http"
 )
 
@@ -73,7 +72,7 @@ func (l *slackLogger) postMessage(r io.Reader) error {
 	defer resp.Body.Close()
 
 	if resp.StatusCode/100 != 2 {
-		data, err := ioutil.ReadAll(resp.Body)
+		data, err := io.ReadAll(resp.Body)
 		if err != nil {
 			return fmt.Errorf("read HTTP response body: %v", err)
 		}

slack_test.go

@@ -3,7 +3,7 @@ package clog
 import (
 	"bytes"
 	"errors"
-	"io/ioutil"
+	"io"
 	"net/http"
 	"testing"
 
@@ -223,7 +223,7 @@ func Test_slackLogger_postMessage(t *testing.T) {
 
 				return &http.Response{
 					StatusCode: statusCode,
-					Body:       ioutil.NopCloser(bytes.NewBufferString(respBody)),
+					Body:       io.NopCloser(bytes.NewBufferString(respBody)),
 					Header:     make(http.Header),
 				}
 			}),