Makefile: 'make fmtchk' prints incorrect output

This should be clearer to the end user about where the problem is.

Makefile

@@ -62,7 +62,11 @@ generate:
 
 .PHONY: fmtcheck
 fmtchk:
-	go run github.com/kevinburke/differ@latest gofmt -l .
+	@OUTPUT=$$(gofmt -l . | grep -v vendor/); \
+		if [ -n "$$OUTPUT" ]; then \
+			echo "Some files were not gofmtted:\\n$$OUTPUT"; \
+			exit 1; \
+		fi
 
 .PHONY: fmtfix
 fmtfix:

pkg/event/entry.go

@@ -2,7 +2,8 @@ package event
 
 // entry represents a single row in the changelog
 // e.g.
-//   {"seq":1,"family":"fam","table":"foo","key":[{"name":"id","type":"int","value":1}]}
+//
+//	{"seq":1,"family":"fam","table":"foo","key":[{"name":"id","type":"int","value":1}]}
 type entry struct {
 	Seq    int64  `json:"seq"`
 	Family string `json:"family"`

pkg/executive/executive.go

@@ -78,7 +78,7 @@ func newMutationRequest(famName schema.FamilyName, req ExecutiveMutationRequest)
 }
 
 // Returns the request Values as a slice in the order specified by the
-//fieldOrder param. An error will be returned if a field is missing.
+// fieldOrder param. An error will be returned if a field is missing.
 func (r *mutationRequest) valuesByOrder(fieldOrder []schema.FieldName) ([]interface{}, error) {
 	values := []interface{}{}
 	for _, fn := range fieldOrder {

pkg/ldbwriter/ldb_writer_with_changelog.go

@@ -19,7 +19,6 @@ type LDBWriterWithChangelog struct {
 	Seq             int64
 }
 
-//
 // NOTE: How does the changelog work?
 //
 // This is sort of the crux of how the changelog comes together. The Reflector
@@ -33,7 +32,6 @@ type LDBWriterWithChangelog struct {
 // This is pretty complex, but after enumerating about 8 different options, it
 // ended up actually being the most simple. Other options involved not-so-great
 // options like parsing SQL or maintaining triggers on every table.
-//
 func (w *LDBWriterWithChangelog) ApplyDMLStatement(ctx context.Context, statement schema.DMLStatement) error {
 	err := w.LdbWriter.ApplyDMLStatement(ctx, statement)
 	if err != nil {

pkg/unsafe/unsafe_test.go

@@ -1,3 +1,4 @@
+//go:build !race
 // +build !race
 
 package unsafe

pkg/utils/interface_slice.go

@@ -6,14 +6,13 @@ import "reflect"
 //
 // - Returns empty slice if no args are passed
 //
-// - For a single argument which is of a slice type, the slice
-//   is converted and returned.
+//   - For a single argument which is of a slice type, the slice
+//     is converted and returned.
 //
-// - For a single argument which is not a slice type, the value is
-//   returned within a single-element slice.
+//   - For a single argument which is not a slice type, the value is
+//     returned within a single-element slice.
 //
 // - For multiple arguments, returns a slice with all the args
-//
 func InterfaceSlice(any ...interface{}) []interface{} {
 	if len(any) == 0 {
 		return []interface{}{}

pkg/version/version_go1_12.go

@@ -1,3 +1,4 @@
+//go:build go1.12
 // +build go1.12
 
 package version

tools.go

@@ -1,3 +1,4 @@
+//go:build tools
 // +build tools
 
 package ctlstore