style: remove redundant comments and fix comment punctuation

Author: ValentinLutz

assert_eventually_test.go

@@ -518,7 +518,7 @@ func TestEventually_ChecksOnceMoreAtTimeout(t *testing.T) {
 	t.Parallel()
 
 	// given: a condition that only becomes true on its second check, and an
-	// interval far larger than the timeout so no ticker check fires; the only
+	// interval far larger than the timeout so no ticker check fires. The only
 	// checks are the immediate one and the deadline one
 	calls := 0
 	mt := &mockT{}

assert_test.go

@@ -1292,7 +1292,7 @@ func TestOrderedComparisons_FailOnNaN(t *testing.T) {
 	t.Run("Between", func(t *testing.T) {
 		t.Parallel()
 
-		// given: a NaN value; when: asserting it lies between bounds
+		// given: a NaN value, when: asserting it lies between bounds
 		mt := &mockT{}
 		testastic.Between(mt, nan, 0.0, 10.0)
 
@@ -1341,7 +1341,7 @@ func TestNil_HandlesUnsafePointer(t *testing.T) {
 
 	var p unsafe.Pointer
 
-	// given: a nil unsafe.Pointer; when/then: Nil passes
+	// given: a nil unsafe.Pointer, when/then: Nil passes
 	mt := &mockT{}
 
 	testastic.Nil(mt, p)
@@ -1366,7 +1366,7 @@ func TestContains_TypedNilStringerDoesNotPanic(t *testing.T) {
 	// given: a typed-nil pointer whose String() dereferences the receiver
 	var s *nilDerefStringer
 
-	// when: asserting Contains on it; then: it reports a failure instead of panicking
+	// when: asserting Contains on it, then: it reports a failure instead of panicking
 	mt := &mockT{}
 
 	testastic.Contains(mt, s, "1")

binary.go

@@ -28,7 +28,7 @@ var errBuildBinaryRequiresImportPath = errors.New("BuildBinary requires importPa
 type Binary struct {
 	path    string
 	workDir string
-	tempDir string // non-empty only for BuildBinaryMain; removed by Cleanup
+	tempDir string // non-empty only for BuildBinaryMain, removed by Cleanup
 }
 
 // RunResult contains the captured stdout, stderr, and exit code from a CLI run.

color_test.go

@@ -107,7 +107,7 @@ func TestDetectColors(t *testing.T) {
 func TestColorize(t *testing.T) {
 	// Save and restore the useColors function so each subtest can stub it.
 	// TestColorize is serial (no t.Parallel), so swapping the variable is
-	// safe — parallel tests in the package only run after this one finishes.
+	// safe, parallel tests in the package only run after this one finishes.
 	savedUseColors := useColors
 
 	t.Cleanup(func() {

diff.go

@@ -26,10 +26,10 @@ func (d diffType) String() string {
 
 // difference represents a single difference between expected and actual JSON.
 type difference struct {
-	Path     string   // JSON path, e.g., "$.users[0].name"
-	Expected any      // Expected value (or matcher description)
-	Actual   any      // Actual value
-	Type     diffType // Type of difference
+	Path     string // JSON path, e.g., "$.users[0].name"
+	Expected any    // Expected value (or matcher description)
+	Actual   any
+	Type     diffType
 }
 
 func formatFileDiffInline(expected, actual []string) string {

file_parse_test.go

@@ -115,7 +115,7 @@ func TestParseLine(t *testing.T) {
 	t.Run("regex matcher", func(t *testing.T) {
 		t.Parallel()
 
-		// given: a line with a regex matcher (no anchors - they're added by parseLine)
+		// given: a line with a regex matcher (no anchors, they're added by parseLine)
 		line := "Email: {{regex `[a-z]+@example\\.com`}}"
 
 		// when: parsing the line

options.go

@@ -31,14 +31,12 @@ type Option func(*config)
 // Shared fields apply to all assertion types. HTML-specific fields
 // are only used by AssertHTML and are ignored by other assertion types.
 type config struct {
-	// Shared fields.
 	IgnoreArrayOrder      bool
 	IgnoreArrayOrderPaths []string
 	IgnoredFields         []string
 	Update                bool
 	Message               string
 
-	// HTML-specific fields.
 	IgnoreComments        bool
 	PreserveWhitespace    bool
 	IgnoredElements       []string

process_coverage.go

@@ -46,7 +46,7 @@ var errCovdataFailed = errors.New("go tool covdata textfmt failed")
 //
 // Without CollectSubprocessCoverage, coverage data is written to per-test temp
 // directories and cleaned up automatically. Processes that use [WithCoverDir]
-// are not affected — their coverage data goes to the specified directory.
+// are not affected. Their coverage data goes to the specified directory.
 func CollectSubprocessCoverage(m *testing.M, outputPath string) int {
 	return collectSubprocessCoverage(m.Run, outputPath)
 }
@@ -81,7 +81,7 @@ func collectSubprocessCoverage(run func() int, outputPath string) int {
 
 // convertProcessCoverage converts binary coverage data under coverDir to a
 // text profile at outputPath using `go tool covdata textfmt`. coverDir may
-// hold coverage files directly or one subdir per subprocess run; both
+// hold coverage files directly or one subdir per subprocess run, both
 // layouts are aggregated.
 func convertProcessCoverage(coverDir string, outputPath string) error {
 	inputDirs, err := collectCoverInputDirs(coverDir)
@@ -119,7 +119,7 @@ func convertProcessCoverage(coverDir string, outputPath string) error {
 }
 
 // collectCoverInputDirs returns the input directories for `go tool covdata`.
-// Per-run subdirs each count as one input; a flat coverDir with files counts
+// Per-run subdirs each count as one input, a flat coverDir with files counts
 // as a single input.
 func collectCoverInputDirs(coverDir string) ([]string, error) {
 	entries, err := os.ReadDir(coverDir)