Samlaf avs 538 make lint and fmt checks as part of ci (#275)

* add workflow to enforce make fmt to be run

* update lint workflow to use same version actions as other workflows

* make fmt

.github/workflows/check-fmt.yml

@@ -0,0 +1,33 @@
+name: check that make fmt was run
+on:
+  push:
+    branches:
+      - master
+  pull_request:
+
+permissions:
+  contents: read
+
+jobs:
+  check-make-fmt:
+    name: check make fmt
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: install go1.21
+        uses: actions/setup-go@v5
+        with:
+          go-version: "1.21"
+
+      - name: Run make fmt
+        run: make fmt
+
+      - name: Check if make fmt generated changes that should be committed
+        run: |
+          if [ -n "$(git status --porcelain)" ]; then
+            echo "Error: make fmt generated changes that should be committed. Please run 'make fmt' and commit the changes."
+            git diff
+            git status
+            exit 1
+          fi

.github/workflows/golangci-lint.yml

@@ -11,12 +11,15 @@ jobs:
     name: Lint
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-go@v4
+      - uses: actions/checkout@v4
+
+      - name: install go 1.21
+        uses: actions/setup-go@v5
         with:
-          go-version: '1.21'
+          go-version: "1.21"
+
       - name: golangci-lint
         uses: golangci/golangci-lint-action@v3
         with:
           version: latest
-          args: --timeout 3m
+          args: --timeout 3m

chainio/clients/avsregistry/writer.go

@@ -37,7 +37,7 @@ type AvsRegistryWriter interface {
 	//  - operatorToAvsRegistrationSigExpiry is the expiry time of the signature
 	//
 	// Deprecated: use RegisterOperator instead.
-	// We will only keep high-level functionality such as RegisterOperator, and low level functionality 
+	// We will only keep high-level functionality such as RegisterOperator, and low level functionality
 	// such as this function should eventually all be done with bindings directly instead.
 	RegisterOperatorInQuorumWithAVSRegistryCoordinator(
 		ctx context.Context,