commit df22bb1b1b193c2d9e1ebc2e614b22830dbb7abc
author Dmytro Shteflyuk <kpumuk@kpumuk.info> Sat Dec 20 13:41:21 2025 -0500
committer Jens Geyer <Jens-G@users.noreply.github.com> Wed Mar 11 08:47:12 2026 +0100
tree f6e11adbd6d1bfdf05f36e12e4b7035463b29652
parent 45b6935a80859016862080f97678ac415e758bed

Added Rubocop harness to enforce Ruby code style

.github/workflows/sca.yml

diff --git a/.github/workflows/sca.yml b/.github/workflows/sca.yml
index 9fa2a16..ae41f86 100644
--- a/.github/workflows/sca.yml
+++ b/.github/workflows/sca.yml
@@ -62,6 +62,13 @@
         run: |
           python -m pip install --upgrade pip setuptools wheel flake8
 
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: "2.7"
+          bundler-cache: true
+          working-directory: "lib/rb"
+
       # Generate thrift files so the static code analysis includes an analysis
       # of the files the thrift compiler spits out.
       - name: Build compiler
@@ -155,6 +162,13 @@
           composer install --quiet
           ./vendor/bin/phpcs
 
+      - name: Run rubocop
+        id: rubocop
+        continue-on-error: true
+        working-directory: "lib/rb"
+        run: |
+          bundle exec rubocop --config .rubocop.yml --format progress --format github . ../../test/rb ../../tutorial/rb
+
       - name: Print statistics
         if: ${{ always() }}
         run: |
@@ -176,6 +190,7 @@
           CPPCHECK_OUTCOME: ${{ steps.cppcheck.outcome }}
           FLAKE8_OUTCOME: ${{ steps.flake8.outcome }}
           PHPCS_OUTCOME: ${{ steps.phpcs.outcome }}
+          RUBOCOP_OUTCOME: ${{ steps.rubocop.outcome }}
         run: |
           failed=0
 
@@ -191,5 +206,9 @@
             echo "::error::Step 'phpcs' failed (outcome: $PHPCS_OUTCOME)"
             failed=1
           fi
+          if [ "$RUBOCOP_OUTCOME" != "success" ]; then
+            echo "::error::Step 'rubocop' failed (outcome: $RUBOCOP_OUTCOME)"
+            failed=1
+          fi
 
           exit $failed