style: make linter happy

Author: zbindenren

config/config.go

@@ -5,7 +5,6 @@ import (
 	"errors"
 	"fmt"
 	"io"
-	"io/ioutil"
 	"os"
 	"path/filepath"
 	"sort"
@@ -114,7 +113,7 @@ func Load(dir string) (*Changelog, error) {
 
 // Read unmarshals config.Changelog from a io.Reader.
 func Read(r io.Reader) (*Changelog, error) {
-	b, err := ioutil.ReadAll(r) // nolint: gosec
+	b, err := io.ReadAll(r)
 	if err != nil {
 		return nil, err
 	}
@@ -137,7 +136,7 @@ func Write(dir string, c Changelog) error {
 		return err
 	}
 
-	if err := ioutil.WriteFile(p, b, 0600); err != nil {
+	if err := os.WriteFile(p, b, 0o600); err != nil {
 		return fmt.Errorf("failed to write file %s: %w", p, err)
 	}
 

internal/changelog/changelog_test.go

@@ -2,7 +2,8 @@ package changelog
 
 import (
 	"bytes"
-	"io/ioutil"
+	"os"
+	"path/filepath"
 	"runtime"
 	"strings"
 	"testing"
@@ -178,7 +179,7 @@ type message struct {
 func messagesFrom(t *testing.T, filePath string) []message {
 	m := []message{}
 
-	d, err := ioutil.ReadFile(filePath) // nolint: gosec
+	d, err := os.ReadFile(filepath.Clean(filePath))
 	require.NoError(t, err)
 
 	err = yaml.Unmarshal(d, &m)

internal/changelog/markdown_notwindows.go

@@ -1,3 +1,4 @@
+//go:build !windows
 // +build !windows
 
 package changelog

internal/cmd/cmd.go

@@ -136,7 +136,7 @@ func (c Command) Run() error {
 	var dst io.Writer = os.Stdout
 
 	if !*c.toStdOut {
-		f, err := os.OpenFile(*c.file, os.O_CREATE|os.O_RDWR, 0600)
+		f, err := os.OpenFile(*c.file, os.O_CREATE|os.O_RDWR, 0o600)
 		if err != nil {
 			return err
 		}

internal/cmd/cmd_test.go

@@ -28,7 +28,7 @@ func TestInit(t *testing.T) {
 	err := c.Run()
 	require.NoError(t, err)
 
-	b, err := ioutil.ReadFile(changlogPath) // nolint: gosec
+	b, err := os.ReadFile(changlogPath) // nolint: gosec
 	require.NoError(t, err)
 
 	expectedFmt := `## 0.1.0 (%s)
@@ -66,7 +66,7 @@ func TestRelease(t *testing.T) {
 	err := c.Run()
 	require.NoError(t, err)
 
-	b, err := ioutil.ReadFile(changlogPath) // nolint: gosec
+	b, err := os.ReadFile(changlogPath) // nolint: gosec
 	require.NoError(t, err)
 
 	expectedFmt := `## 0.2.0 (%s)

internal/cmd/release.go

@@ -4,7 +4,6 @@ import (
 	"errors"
 	"fmt"
 	"io"
-	"io/ioutil"
 	"os"
 	"time"
 
@@ -79,7 +78,7 @@ func (c Command) runRelease(dst io.Writer, l *flash.Logger, cfg config.Changelog
 
 	title := fmt.Sprintf("%s (%s)", version, time.Now().Format(dateFormat))
 
-	old, err := ioutil.ReadFile(*c.file)
+	old, err := os.ReadFile(*c.file)
 	if err != nil && !*c.toStdOut {
 		return err
 	}

lex.go

@@ -289,7 +289,8 @@ func peekString(l *lexer.L, count int) string {
 		}
 
 		s += string(l.Next())
-		defer l.Rewind()
+
+		defer l.Rewind() //nolint: gocritic
 	}
 
 	return s

parse_test.go

@@ -3,7 +3,7 @@ package cc
 import (
 	"encoding/json"
 	"fmt"
-	"io/ioutil"
+	"os"
 	"testing"
 
 	"github.com/stretchr/testify/require"
@@ -27,7 +27,7 @@ func TestParse(t *testing.T) {
 	}
 
 	allTests := tests{}
-	d, err := ioutil.ReadFile("tests.yaml")
+	d, err := os.ReadFile("tests.yaml")
 	require.NoError(t, err)
 
 	err = yaml.Unmarshal(d, &allTests)