DAOS-16263 cq: merge yamllint and clang-format into linting

merge yamllint and clang-format into linting workflow so all lint checks
are grouped together.

Make yaml-lint required but clang-format optional until stable.

Skip-test: true
Skip-build: true

Required-githooks: true

Signed-off-by: Dalton Bohning <dalton.bohning@intel.com>

.github/workflows/linting.yml

@@ -161,10 +161,56 @@ jobs:
           ignore_words_file: ci/codespell.ignores
           builtin: clear,rare,informal,names,en-GB_to_en-US
 
+  clang-format:
+    name: Clang Format
+    runs-on: ubuntu-22.04
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+      - name: Pull via git
+        run: git fetch origin ${{ github.event.pull_request.base.ref }}
+      - name: Run check in docker
+        uses: ./.github/actions/clang-format
+        with:
+          target: origin/${{ github.event.pull_request.base.ref }}
+      - name: Export changes
+        uses: actions/upload-artifact@v4
+        if: failure()
+        with:
+          name: format-patch-for-pr-${{ github.event.pull_request.number }}
+          path: auto-format-changes.diff
+
+  yaml-lint:
+    name: Yamllint check
+    runs-on: ubuntu-22.04
+    steps:
+      - name: Check out source repository
+        uses: actions/checkout@v4
+        with:
+          ref: ${{ github.event.pull_request.head.sha }}
+      - name: Set up Python environment
+        uses: actions/setup-python@v5
+        with:
+          python-version: '3'
+      - name: Install extra python packages
+        run: pip install --requirement utils/cq/requirements.txt
+      - name: Run check
+        run: yamllint --format github .
+
   linting-summary:
     name: Linting Summary
     runs-on: ubuntu-22.04
-    needs: [isort, shell-check, log-check, ftest-tags, flake8-lint, doxygen, pylint, codespell]
+    needs:
+      - isort
+      - shell-check
+      - log-check
+      - ftest-tags
+      - flake8-lint
+      - doxygen
+      - pylint
+      - codespell
+      # - clang-format  # not required
+      - yaml-lint
     if: (!cancelled())
     steps:
       - name: Check if any job failed