fix: lint issues (#280)

caarlos0 · web-flow · commit ef3b2d1936f3 · 2024-03-08T16:32:19.000-03:00
Signed-off-by: Carlos Alexandro Becker &lt;caarlos0@users.noreply.github.com&gt;
diff --git a/ansi/codeblock.go b/ansi/codeblock.go
@@ -16,11 +16,9 @@ const (
 	chromaStyleTheme = "charm"
 )
 
-var (
-	// mutex for synchronizing access to the chroma style registry.
-	// Related https://github.com/alecthomas/chroma/pull/650
-	mutex = sync.Mutex{}
-)
+// mutex for synchronizing access to the chroma style registry.
+// Related https://github.com/alecthomas/chroma/pull/650
+var mutex = sync.Mutex{}
 
 // A CodeBlockElement is used to render code blocks.
 type CodeBlockElement struct {
diff --git a/ansi/renderer_test.go b/ansi/renderer_test.go
@@ -3,7 +3,7 @@ package ansi
 import (
 	"bytes"
 	"encoding/json"
-	"io/ioutil"
+	"os"
 	"path/filepath"
 	"strings"
 	"testing"
@@ -35,11 +35,11 @@ func TestRenderer(t *testing.T) {
 		sn := filepath.Join(examplesDir, bn+".style")
 		tn := filepath.Join("../testdata", bn+".test")
 
-		in, err := ioutil.ReadFile(f)
+		in, err := os.ReadFile(f)
 		if err != nil {
 			t.Fatal(err)
 		}
-		b, err := ioutil.ReadFile(sn)
+		b, err := os.ReadFile(sn)
 		if err != nil {
 			t.Fatal(err)
 		}
@@ -77,15 +77,15 @@ func TestRenderer(t *testing.T) {
 
 		// generate
 		if generateExamples {
-			err = ioutil.WriteFile(tn, buf.Bytes(), 0644)
+			err = os.WriteFile(tn, buf.Bytes(), 0644)
 			if err != nil {
 				t.Fatal(err)
 			}
 			continue
 		}
 
 		// verify
-		td, err := ioutil.ReadFile(tn)
+		td, err := os.ReadFile(tn)
 		if err != nil {
 			t.Fatal(err)
 		}
@@ -108,11 +108,11 @@ func TestRendererIssues(t *testing.T) {
 		t.Run(bn, func(t *testing.T) {
 			tn := filepath.Join(issuesDir, bn+".test")
 
-			in, err := ioutil.ReadFile(f)
+			in, err := os.ReadFile(f)
 			if err != nil {
 				t.Fatal(err)
 			}
-			b, err := ioutil.ReadFile("../styles/dark.json")
+			b, err := os.ReadFile("../styles/dark.json")
 			if err != nil {
 				t.Fatal(err)
 			}
@@ -150,15 +150,15 @@ func TestRendererIssues(t *testing.T) {
 
 			// generate
 			if generateIssues {
-				err = ioutil.WriteFile(tn, buf.Bytes(), 0644)
+				err = os.WriteFile(tn, buf.Bytes(), 0644)
 				if err != nil {
 					t.Fatal(err)
 				}
 				return
 			}
 
 			// verify
-			td, err := ioutil.ReadFile(tn)
+			td, err := os.ReadFile(tn)
 			if err != nil {
 				t.Fatal(err)
 			}
diff --git a/glamour_test.go b/glamour_test.go
@@ -2,7 +2,8 @@ package glamour
 
 import (
 	"bytes"
-	"io/ioutil"
+	"io"
+	"os"
 	"strings"
 	"testing"
 )
@@ -21,7 +22,7 @@ func TestTermRendererWriter(t *testing.T) {
 		t.Fatal(err)
 	}
 
-	in, err := ioutil.ReadFile(markdown)
+	in, err := os.ReadFile(markdown)
 	if err != nil {
 		t.Fatal(err)
 	}
@@ -35,22 +36,22 @@ func TestTermRendererWriter(t *testing.T) {
 		t.Fatal(err)
 	}
 
-	b, err := ioutil.ReadAll(r)
+	b, err := io.ReadAll(r)
 	if err != nil {
 		t.Fatal(err)
 	}
 
 	// generate
 	if generate {
-		err := ioutil.WriteFile(testFile, b, 0644)
+		err := os.WriteFile(testFile, b, 0644)
 		if err != nil {
 			t.Fatal(err)
 		}
 		return
 	}
 
 	// verify
-	td, err := ioutil.ReadFile(testFile)
+	td, err := os.ReadFile(testFile)
 	if err != nil {
 		t.Fatal(err)
 	}
@@ -69,7 +70,7 @@ func TestTermRenderer(t *testing.T) {
 		t.Fatal(err)
 	}
 
-	in, err := ioutil.ReadFile(markdown)
+	in, err := os.ReadFile(markdown)
 	if err != nil {
 		t.Fatal(err)
 	}
@@ -80,7 +81,7 @@ func TestTermRenderer(t *testing.T) {
 	}
 
 	// verify
-	td, err := ioutil.ReadFile(testFile)
+	td, err := os.ReadFile(testFile)
 	if err != nil {
 		t.Fatal(err)
 	}
@@ -121,7 +122,7 @@ func TestWithPreservedNewLines(t *testing.T) {
 		t.Fatal(err)
 	}
 
-	in, err := ioutil.ReadFile("testdata/preserved_newline.in")
+	in, err := os.ReadFile("testdata/preserved_newline.in")
 	if err != nil {
 		t.Fatal(err)
 	}
@@ -132,7 +133,7 @@ func TestWithPreservedNewLines(t *testing.T) {
 	}
 
 	// verify
-	td, err := ioutil.ReadFile("testdata/preserved_newline.test")
+	td, err := os.ReadFile("testdata/preserved_newline.test")
 	if err != nil {
 		t.Fatal(err)
 	}
@@ -167,7 +168,7 @@ func TestStyles(t *testing.T) {
 }
 
 func TestRenderHelpers(t *testing.T) {
-	in, err := ioutil.ReadFile(markdown)
+	in, err := os.ReadFile(markdown)
 	if err != nil {
 		t.Fatal(err)
 	}
@@ -178,7 +179,7 @@ func TestRenderHelpers(t *testing.T) {
 	}
 
 	// verify
-	td, err := ioutil.ReadFile(testFile)
+	td, err := os.ReadFile(testFile)
 	if err != nil {
 		t.Fatal(err)
 	}
@@ -209,7 +210,7 @@ func TestCapitalization(t *testing.T) {
 	}
 
 	// expected outcome
-	td, err := ioutil.ReadFile("testdata/capitalization.test")
+	td, err := os.ReadFile("testdata/capitalization.test")
 	if err != nil {
 		t.Fatal(err)
 	}

Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,8 @@ package glamour`
`2`	`2`
`3`	`3`	`import (`
`4`	`4`	`"bytes"`
`5`		`- "io/ioutil"`
	`5`	`+ "io"`
	`6`	`+ "os"`
`6`	`7`	`"strings"`
`7`	`8`	`"testing"`
`8`	`9`	`)`
`@@ -21,7 +22,7 @@ func TestTermRendererWriter(t *testing.T) {`
`21`	`22`	`t.Fatal(err)`
`22`	`23`	`}`
`23`	`24`
`24`		`- in, err := ioutil.ReadFile(markdown)`
	`25`	`+ in, err := os.ReadFile(markdown)`
`25`	`26`	`if err != nil {`
`26`	`27`	`t.Fatal(err)`
`27`	`28`	`}`
`@@ -35,22 +36,22 @@ func TestTermRendererWriter(t *testing.T) {`
`35`	`36`	`t.Fatal(err)`
`36`	`37`	`}`
`37`	`38`
`38`		`- b, err := ioutil.ReadAll(r)`
	`39`	`+ b, err := io.ReadAll(r)`
`39`	`40`	`if err != nil {`
`40`	`41`	`t.Fatal(err)`
`41`	`42`	`}`
`42`	`43`
`43`	`44`	`// generate`
`44`	`45`	`if generate {`
`45`		`- err := ioutil.WriteFile(testFile, b, 0644)`
	`46`	`+ err := os.WriteFile(testFile, b, 0644)`
`46`	`47`	`if err != nil {`
`47`	`48`	`t.Fatal(err)`
`48`	`49`	`}`
`49`	`50`	`return`
`50`	`51`	`}`
`51`	`52`
`52`	`53`	`// verify`
`53`		`- td, err := ioutil.ReadFile(testFile)`
	`54`	`+ td, err := os.ReadFile(testFile)`
`54`	`55`	`if err != nil {`
`55`	`56`	`t.Fatal(err)`
`56`	`57`	`}`
`@@ -69,7 +70,7 @@ func TestTermRenderer(t *testing.T) {`
`69`	`70`	`t.Fatal(err)`
`70`	`71`	`}`
`71`	`72`
`72`		`- in, err := ioutil.ReadFile(markdown)`
	`73`	`+ in, err := os.ReadFile(markdown)`
`73`	`74`	`if err != nil {`
`74`	`75`	`t.Fatal(err)`
`75`	`76`	`}`
`@@ -80,7 +81,7 @@ func TestTermRenderer(t *testing.T) {`
`80`	`81`	`}`
`81`	`82`
`82`	`83`	`// verify`
`83`		`- td, err := ioutil.ReadFile(testFile)`
	`84`	`+ td, err := os.ReadFile(testFile)`
`84`	`85`	`if err != nil {`
`85`	`86`	`t.Fatal(err)`
`86`	`87`	`}`
`@@ -121,7 +122,7 @@ func TestWithPreservedNewLines(t *testing.T) {`
`121`	`122`	`t.Fatal(err)`
`122`	`123`	`}`
`123`	`124`
`124`		`- in, err := ioutil.ReadFile("testdata/preserved_newline.in")`
	`125`	`+ in, err := os.ReadFile("testdata/preserved_newline.in")`
`125`	`126`	`if err != nil {`
`126`	`127`	`t.Fatal(err)`
`127`	`128`	`}`
`@@ -132,7 +133,7 @@ func TestWithPreservedNewLines(t *testing.T) {`
`132`	`133`	`}`
`133`	`134`
`134`	`135`	`// verify`
`135`		`- td, err := ioutil.ReadFile("testdata/preserved_newline.test")`
	`136`	`+ td, err := os.ReadFile("testdata/preserved_newline.test")`
`136`	`137`	`if err != nil {`
`137`	`138`	`t.Fatal(err)`
`138`	`139`	`}`
`@@ -167,7 +168,7 @@ func TestStyles(t *testing.T) {`
`167`	`168`	`}`
`168`	`169`
`169`	`170`	`func TestRenderHelpers(t *testing.T) {`
`170`		`- in, err := ioutil.ReadFile(markdown)`
	`171`	`+ in, err := os.ReadFile(markdown)`
`171`	`172`	`if err != nil {`
`172`	`173`	`t.Fatal(err)`
`173`	`174`	`}`
`@@ -178,7 +179,7 @@ func TestRenderHelpers(t *testing.T) {`
`178`	`179`	`}`
`179`	`180`
`180`	`181`	`// verify`
`181`		`- td, err := ioutil.ReadFile(testFile)`
	`182`	`+ td, err := os.ReadFile(testFile)`
`182`	`183`	`if err != nil {`
`183`	`184`	`t.Fatal(err)`
`184`	`185`	`}`
`@@ -209,7 +210,7 @@ func TestCapitalization(t *testing.T) {`
`209`	`210`	`}`
`210`	`211`
`211`	`212`	`// expected outcome`
`212`		`- td, err := ioutil.ReadFile("testdata/capitalization.test")`
	`213`	`+ td, err := os.ReadFile("testdata/capitalization.test")`
`213`	`214`	`if err != nil {`
`214`	`215`	`t.Fatal(err)`
`215`	`216`	`}`