Address yamllint findings

Signed-off-by: David Son <[email protected]>

Author: sondavidb

.github/ISSUE_TEMPLATE/bug_report.yaml

@@ -33,4 +33,4 @@ body:
 
   - type: textarea
     attributes:
-      label: Any additional context or information about the bug
+      label: Any additional context or information about the bug

.github/ISSUE_TEMPLATE/config.yml

@@ -6,4 +6,4 @@ contact_links:
       In most cases, GitHub Discussions is the preferred place to ask a question, 
       and visible to more users. If you encountered a bug or request a feature, 
       please use the issue template. Otherwise, please consider asking in GitHub
-      Discussions first.
+      Discussions first.

.github/ISSUE_TEMPLATE/feature_request.yaml

@@ -21,4 +21,4 @@ body:
 
   - type: textarea
     attributes:
-      label: Any additional context or information about the feature request
+      label: Any additional context or information about the feature request

.github/dependabot.yml

@@ -1,14 +1,15 @@
 version: 2
 updates:
- 
+
   # Automatic upgrade for go modules.
   - package-ecosystem: "gomod"
     directory: "/"
     schedule:
       interval: "daily"
     ignore:
-      # skip grpc because the current latest is not compatible with containerd 1.7
-      # skip k8s deps since they use the latest go version/features that may not be in the go version soci uses
+      # skip grpc because the current latest not compatible with containerd 1.7
+      # skip k8s deps since they use the latest go version/features that may
+      # not be in the go version soci uses
       # Also ignored in /scripts/bump-deps.sh
       - dependency-name: "google.golang.org/grpc"
       - dependency-name: "k8s.io/*"
@@ -19,8 +20,9 @@ updates:
     schedule:
       interval: "daily"
     ignore:
-      # skip grpc because the current latest is not compatible with containerd 1.7
-      # skip k8s deps since they use the latest go version/features that may not be in the go version soci uses
+      # skip grpc because the current latest not compatible with containerd 1.7
+      # skip k8s deps since they use the latest go version/features that may
+      # not be in the go version soci uses
       # Also ignored in /scripts/bump-deps.sh
       - dependency-name: "github.com/awslabs/soci-snapshotter"
       - dependency-name: "google.golang.org/grpc"

.github/workflows/benchmark_visualization.yml

@@ -2,7 +2,7 @@ name: Visualize Benchmark and upload results
 
 on:
   push:
-    branches: [ main ]
+    branches: [main]
     paths:
       - '**'
       - '!docs/**' # ignore docs changes
@@ -14,7 +14,7 @@ permissions:
 
 env:
   GO_VERSION: '1.21.9'
-      
+
 jobs:
   benchmark:
     runs-on: ubuntu-20.04
@@ -29,13 +29,13 @@ jobs:
           go-version: ${{ env.GO_VERSION }}
       - run: make
       - name: Run benchmark
-        run: make benchmarks-perf-test 
+        run: make benchmarks-perf-test
       - name: Upload latest benchmark result
         uses: actions/upload-artifact@v4
         with:
           name: benchmark-result-artifact
           path: ${{github.workspace}}/benchmark/performanceTest/output/results.json
-      
+
   download-and-convert-benchmark-result-to-visualization-data:
     runs-on: ubuntu-20.04
     needs: benchmark
@@ -52,10 +52,10 @@ jobs:
       - name: Change permission of visualization-data-converter.sh
         run: chmod +x ${{github.workspace}}/scripts/visualization-data-converter.sh
       - name: Convert benchmark result to visualization data
-        run: ${{github.workspace}}/scripts/visualization-data-converter.sh ${{github.workspace}}/results.json ${{github.workspace}}/current  
+        run: ${{github.workspace}}/scripts/visualization-data-converter.sh ${{github.workspace}}/results.json ${{github.workspace}}/current
       - name: Build matrix of filepaths
         id: set-matrix
-         # Create a JSON array with the file paths and store it in the 'files' output
+        # Create a JSON array with the file paths and store it in the 'files' output
         run: echo "files=$(find ${{github.workspace}}/current -type f -name '*.json' -printf '%p\n' | jq -R . | jq -cs .)" >> $GITHUB_OUTPUT
       - name: Upload visualization files as github artifacts
         uses: actions/upload-artifact@v4
@@ -84,15 +84,15 @@ jobs:
         with:
           name: benchmark-gh-pages-artifact
           path: ${{github.workspace}}/current
-      
+
       - name: Extract file names
         id: get_filename
         run: |
           # Get the filename without the path
           filename=$(basename "${{ matrix.file }}")
           # Remove the .json extension
           filename_without_extension="${filename%.json}"
-          echo "filename=$filename_without_extension" >> $GITHUB_OUTPUT    
+          echo "filename=$filename_without_extension" >> $GITHUB_OUTPUT
 
       - name: Run benchmark action
         uses: benchmark-action/github-action-benchmark@v1

.github/workflows/bump-deps.yml

@@ -4,7 +4,7 @@ name: Bump Deps
 
 on:
   schedule:
-  - cron: '0 10 * * 2' # weekly at 10AM Tuesday
+    - cron: '0 10 * * 2' # weekly at 10AM Tuesday
   workflow_dispatch:
 
 env:
@@ -23,19 +23,19 @@ jobs:
     runs-on: ubuntu-20.04
 
     steps:
-    - uses: actions/checkout@v4
-    - uses: actions/setup-go@v5
-      with:
-        go-version: ${{ env.GO_VERSION }}
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version: ${{ env.GO_VERSION }}
 
-    - run: ./scripts/bump-deps.sh
-    - name: Create Pull Request
-      uses: peter-evans/create-pull-request@v6
-      with:
-        title: "Bump dependencies using scripts/bump-deps.sh"
-        commit-message: "Bump dependencies using scripts/bump-deps.sh"
-        body: "This PR created by [create-pull-request](https://github.com/peter-evans/create-pull-request) must be closed and reopened manually to trigger automated checks."
-        labels: dependencies
-        delete-branch: true
-        author: "GitHub <[email protected]>"
-        signoff: true
+      - run: ./scripts/bump-deps.sh
+      - name: Create Pull Request
+        uses: peter-evans/create-pull-request@v6
+        with:
+          title: "Bump dependencies using scripts/bump-deps.sh"
+          commit-message: "Bump dependencies using scripts/bump-deps.sh"
+          body: "This PR created by [create-pull-request](https://github.com/peter-evans/create-pull-request) must be closed and reopened manually to trigger automated checks."
+          labels: dependencies
+          delete-branch: true
+          author: "GitHub <[email protected]>"
+          signoff: true

.github/workflows/prebuild.yml

@@ -2,9 +2,9 @@ name: Pre-build
 
 on:
   push:
-    branches: [ 'main', 'release/**']
+    branches: ['main', 'release/**']
   pull_request:
-    branches: [ 'main', 'release/**']
+    branches: ['main', 'release/**']
 
 env:
   GO_VERSION: '1.21.9'
@@ -37,10 +37,10 @@ jobs:
       - name: Pull latest awslabs/git-secrets repo
         uses: actions/checkout@v4
         with:
-           repository: awslabs/git-secrets
-           ref: 1.3.0
-           fetch-tags: true
-           path: git-secrets
+          repository: awslabs/git-secrets
+          ref: 1.3.0
+          fetch-tags: true
+          path: git-secrets
       - name: Install git secrets from source
         run: sudo make install
         working-directory: git-secrets
@@ -49,7 +49,7 @@ jobs:
         run: |
           git secrets --register-aws
           git secrets --scan-history
-  
+
   lint:
     strategy:
       matrix:

.github/workflows/releases.yml

@@ -20,30 +20,30 @@ jobs:
       DYNAMIC_BINARY_NAME: ''
       STATIC_BINARY_NAME: ''
     steps:
-    - uses: actions/checkout@v4
-    - name: Setup and export variables
-      run: |
-        export release_tag=${GITHUB_REF#refs/*/} # Strip down to raw tag name
-        export release_version=${release_tag/v/} # Remove v from tag name
+      - uses: actions/checkout@v4
+      - name: Setup and export variables
+        run: |
+          export release_tag=${GITHUB_REF#refs/*/} # Strip down to raw tag name
+          export release_version=${release_tag/v/} # Remove v from tag name
 
-        echo "RELEASE_TAG=${release_tag}" >> $GITHUB_ENV
-        echo "DYNAMIC_BINARY_NAME=soci-snapshotter-${release_version}-linux-amd64.tar.gz" >> $GITHUB_ENV
-        echo "STATIC_BINARY_NAME=soci-snapshotter-${release_version}-linux-amd64-static.tar.gz" >> $GITHUB_ENV
-        
-        mkdir release
-    - name: Create release binaries
-      run: make RELEASE_TAG=${{ env.RELEASE_TAG }} release
-    - uses: actions/upload-artifact@v4
-      with:
-        name: artifacts
-        path: release/
-        if-no-files-found: error
+          echo "RELEASE_TAG=${release_tag}" >> $GITHUB_ENV
+          echo "DYNAMIC_BINARY_NAME=soci-snapshotter-${release_version}-linux-amd64.tar.gz" >> $GITHUB_ENV
+          echo "STATIC_BINARY_NAME=soci-snapshotter-${release_version}-linux-amd64-static.tar.gz" >> $GITHUB_ENV
+
+          mkdir release
+      - name: Create release binaries
+        run: make RELEASE_TAG=${{ env.RELEASE_TAG }} release
+      - uses: actions/upload-artifact@v4
+        with:
+          name: artifacts
+          path: release/
+          if-no-files-found: error
 
     outputs:
       release_tag: ${{ env.RELEASE_TAG }}
       dynamic_binary_name: ${{ env.DYNAMIC_BINARY_NAME }}
       static_binary_name: ${{ env.STATIC_BINARY_NAME }}
-  
+
   validate-artifacts:
     needs: generate-artifacts
     runs-on: ubuntu-20.04

.golangci.yml

@@ -18,8 +18,9 @@ issues:
   exclude-rules:
     - linters:
         - revive
-      # Ignore unused parameter rule. It's not enforced by go and it can make it hard
-      # to understand what the unused parameter was supposed to be used for.
+      # Ignore unused parameter rule. It's not enforced by go and
+      # it can make it hard to understand what the unused parameter
+      # was supposed to be used for.
       text: "unused-parameter:"
 
 run: