ci: migrate checking by rflint from TravisCI to GitHub Actions

Part of #1154

Author: php-coder

.github/workflows/quality-checks.yml

@@ -96,3 +96,21 @@ jobs:
             [email protected]
       - name: Run bootlint
         run: ./src/main/scripts/execute-command.sh bootlint
+  run-rflint:
+    name: Run RobotFramework linter
+    # https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idruns-on
+    runs-on: ubuntu-20.04
+    steps:
+      - name: Clone source code
+        uses: actions/[email protected] # https://github.com/actions/checkout
+        with:
+          # Whether to configure the token or SSH key with the local git config. Default: true
+          persist-credentials: false
+      - name: Install rflint
+        run: >-
+          pip3 install \
+            --user \
+            robotframework==3.2.2 \
+            robotframework-lint==1.1
+      - name: Run rflint
+        run: ./src/main/scripts/execute-command.sh rflint

.travis.yml

@@ -26,8 +26,6 @@ before_script:
       curl https://bootstrap.pypa.io/pip/3.6/get-pip.py | sudo -H python3.6;
       pip3 install --user \
         html5validator==0.4.2 \
-        robotframework==3.2.2 \
-        robotframework-lint==1.1 \
         ansible==2.9.2 \
         ansible-lint==4.2.0;
       if [ "$TRAVIS_PULL_REQUEST" != 'false' ]; then

src/main/scripts/ci/check-build-and-verify.sh

@@ -58,7 +58,6 @@ if [ "${1:-}" = '--only-integration-tests' ]; then
 	RUN_ONLY_INTEGRATION_TESTS=yes
 fi
 
-RFLINT_STATUS=
 SHELLCHECK_STATUS=
 JEST_STATUS=
 HTML_STATUS=
@@ -74,7 +73,6 @@ if [ "${SPRING_PROFILES_ACTIVE:-}" = 'travis' ] && [ "${TRAVIS_PULL_REQUEST:-fal
 	DANGER_STATUS=
 fi
 
-RFLINT_TIME=0
 SHELLCHECK_TIME=0
 JEST_TIME=0
 HTML_TIME=0
@@ -110,7 +108,6 @@ if [ "$RUN_ONLY_INTEGRATION_TESTS" = 'no' ]; then
 			AFFECTS_JS_FILES="$(echo "$MODIFIED_FILES"      |  grep -q '\.js$' || echo 'no')"
 			AFFECTS_HTML_FILES="$(echo "$MODIFIED_FILES"    |  grep -q '\.html$' || echo 'no')"
 			AFFECTS_JAVA_FILES="$(echo "$MODIFIED_FILES"    |  grep -q '\.java$' || echo 'no')"
-			AFFECTS_ROBOT_FILES="$(echo "$MODIFIED_FILES"   |  grep -q '\.robot$' || echo 'no')"
 			AFFECTS_SHELL_FILES="$(echo "$MODIFIED_FILES"   |  grep -q '\.sh$' || echo 'no')"
 			AFFECTS_GROOVY_FILES="$(echo "$MODIFIED_FILES"  |  grep -q '\.groovy$' || echo 'no')"
 			AFFECTS_PLAYBOOKS="$(echo "$MODIFIED_FILES"      |  grep -Eq '(vagrant|prod|deploy|bootstrap|/roles/.+)\.yml$' || echo 'no')"
@@ -134,7 +131,6 @@ if [ "$RUN_ONLY_INTEGRATION_TESTS" = 'no' ]; then
 				if [ "$AFFECTS_HTML_FILES" = 'no' ]; then
 					HTML_STATUS=skip
 				fi
-				[ "$AFFECTS_ROBOT_FILES" != 'no' ] || RFLINT_STATUS=skip
 				[ "$AFFECTS_SHELL_FILES" != 'no' ] || SHELLCHECK_STATUS=skip
 
 				if [ "$AFFECTS_PLAYBOOKS" = 'no' ]; then
@@ -154,13 +150,6 @@ if [ "$RUN_ONLY_INTEGRATION_TESTS" = 'no' ]; then
 
 	echo
 
-	if [ "$RFLINT_STATUS" != 'skip' ]; then
-		START_TIME=$SECONDS
-		"$EXEC_CMD" rflint >rflint.log 2>&1 || RFLINT_STATUS=fail
-		RFLINT_TIME=$((SECONDS - START_TIME))
-	fi
-	print_status "$RFLINT_STATUS" "$RFLINT_TIME" 'Run robot framework lint'
-	
 	if [ "$SHELLCHECK_STATUS" != 'skip' ]; then
 		START_TIME=$SECONDS
 		"$EXEC_CMD" shellcheck >shellcheck.log 2>&1 || SHELLCHECK_STATUS=fail
@@ -234,7 +223,6 @@ fi
 print_status "$DANGER_STATUS" "$DANGER_TIME" 'Run danger'
 
 if [ "$RUN_ONLY_INTEGRATION_TESTS" = 'no' ]; then
-	[ "$RFLINT_STATUS" = 'skip' ]         || print_log rflint.log         'Run robot framework lint'
 	[ "$SHELLCHECK_STATUS" = 'skip' ]     || print_log shellcheck.log     'Run shellcheck'
 	[ "$JEST_STATUS" = 'skip' ]           || print_log jest.log           'Run JavaScript unit tests'
 	[ "$HTML_STATUS" = 'skip' ]           || print_log validator.log      'Run html5validator'
@@ -251,8 +239,8 @@ if [ "$DANGER_STATUS" != 'skip' ]; then
 	print_log danger.log 'Run danger'
 fi
 
-rm -f rflint.log shellcheck.log jest.log validator.log enforcer.log test.log codenarc.log spotbugs.log verify.log danger.log ansible_lint.log
+rm -f shellcheck.log jest.log validator.log enforcer.log test.log codenarc.log spotbugs.log verify.log danger.log ansible_lint.log
 
-if echo "$RFLINT_STATUS$SHELLCHECK_STATUS$JEST_STATUS$HTML_STATUS$ENFORCER_STATUS$TEST_STATUS$CODENARC_STATUS$SPOTBUGS_STATUS$VERIFY_STATUS$DANGER_STATUS$ANSIBLE_LINT_STATUS" | grep -Fqs 'fail'; then
+if echo "$SHELLCHECK_STATUS$JEST_STATUS$HTML_STATUS$ENFORCER_STATUS$TEST_STATUS$CODENARC_STATUS$SPOTBUGS_STATUS$VERIFY_STATUS$DANGER_STATUS$ANSIBLE_LINT_STATUS" | grep -Fqs 'fail'; then
 	exit 1
 fi