ci: rename some jobs

ryboe · Nov 9, 2024 · ffeda21 · ffeda21
1 parent 80d900d
commit ffeda21
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/.github/workflows/pr.yml b/.github/workflows/pr.yml
@@ -23,7 +23,7 @@ jobs:
           version: latest
 
   check:
-    name: go vet
+    name: go vet and staticcheck
     runs-on: ubuntu-latest
     steps:
       - name: checkout repository
@@ -42,7 +42,7 @@ jobs:
           version: latest
 
   tests:
-    name: run unit tests with the race detector enabled
+    name: unit tests with race detector enabled
     runs-on: ubuntu-latest
     steps:
       - name: checkout repository

diff --git a/args_test.go b/args_test.go
@@ -167,9 +167,11 @@ func TestExtractingArgsFromSourceText(t *testing.T) {
 		},
 	}
 
+	// We can test both exprToString() and argName() with the test cases above.
 	for _, tc := range testCases {
 		// test exprToString()
 		testName := fmt.Sprintf("exprToString(%T)", tc.arg)
+		// nolint: thelper
 		t.Run(testName, func(t *testing.T) {
 			if _, ok := tc.arg.(*ast.Ident); ok {
 				return
@@ -182,6 +184,7 @@ func TestExtractingArgsFromSourceText(t *testing.T) {
 
 		// test argName()
 		testName = fmt.Sprintf("argName(%T)", tc.arg)
+		// nolint: thelper
 		t.Run(testName, func(t *testing.T) {
 			if got := argName(tc.arg); got != tc.want {
 				t.Fatalf("\ngot:  %s\nwant: %s", got, tc.want)