style: Improve actions and workflows style (#2060)

.github/actions/build_clio/action.yml

@@ -23,4 +23,7 @@ runs:
       shell: bash
       run: |
         cd build
-        cmake --build . --parallel ${{ steps.number_of_threads.outputs.threads_number }} --target ${{ inputs.targets }}
+        cmake \
+          --build . \
+          --parallel ${{ steps.number_of_threads.outputs.threads_number }} \
+          --target ${{ inputs.targets }}

.github/actions/code_coverage/action.yml

@@ -1,5 +1,6 @@
 name: Generate code coverage report
 description: Run tests, generate code coverage report and upload it to codecov.io
+
 runs:
   using: composite
   steps:

.github/actions/create_issue/action.yml

@@ -1,5 +1,6 @@
 name: Create an issue
 description: Create an issue
+
 inputs:
   title:
     description: Issue title
@@ -15,10 +16,12 @@ inputs:
     description: Comma-separated list of assignees
     required: true
     default: "godexsoft,kuznetsss,PeterChen13579"
+
 outputs:
   created_issue_id:
     description: Created issue id
     value: ${{ steps.create_issue.outputs.created_issue }}
+
 runs:
   using: composite
   steps:
@@ -27,7 +30,12 @@ runs:
       shell: bash
       run: |
         echo -e '${{ inputs.body }}' > issue.md
-        gh issue create --assignee '${{ inputs.assignees }}' --label '${{ inputs.labels }}' --title '${{ inputs.title }}' --body-file ./issue.md > create_issue.log
+        gh issue create \
+          --assignee '${{ inputs.assignees }}' \
+          --label '${{ inputs.labels }}' \
+          --title '${{ inputs.title }}' \
+          --body-file ./issue.md \
+          > create_issue.log
         created_issue=$(cat create_issue.log | sed 's|.*/||')
         echo "created_issue=$created_issue" >> $GITHUB_OUTPUT
         rm create_issue.log issue.md

.github/actions/generate/action.yml

@@ -1,5 +1,6 @@
 name: Run conan and cmake
 description: Run conan and cmake
+
 inputs:
   conan_profile:
     description: Conan profile name
@@ -27,7 +28,13 @@ inputs:
   sanitizer:
     description: Sanitizer to use
     required: true
-    default: "false" # false, tsan, asan or ubsan
+    default: "false"
+    choices:
+      - "false"
+      - "tsan"
+      - "asan"
+      - "ubsan"
+
 runs:
   using: composite
   steps:
@@ -44,7 +51,17 @@ runs:
         INTEGRATION_TESTS_OPTION: "${{ inputs.build_integration_tests == 'true' && 'True' || 'False' }}"
       run: |
         cd build
-        conan install .. -of . -b $BUILD_OPTION -s build_type=${{ inputs.build_type }} -o clio:static="${STATIC_OPTION}" -o clio:tests=True -o clio:integration_tests="${INTEGRATION_TESTS_OPTION}" -o clio:lint=False -o clio:coverage="${CODE_COVERAGE}" --profile ${{ inputs.conan_profile }}
+        conan \
+          install .. \
+          -of . \
+          -b $BUILD_OPTION \
+          -s build_type=${{ inputs.build_type }} \
+          -o clio:static="${STATIC_OPTION}" \
+          -o clio:tests=True \
+          -o clio:integration_tests="${INTEGRATION_TESTS_OPTION}" \
+          -o clio:lint=False \
+          -o clio:coverage="${CODE_COVERAGE}" \
+          --profile ${{ inputs.conan_profile }}
 
     - name: Run cmake
       shell: bash
@@ -57,4 +74,9 @@ runs:
               '' }}
       run: |
         cd build
-        cmake -DCMAKE_TOOLCHAIN_FILE:FILEPATH=build/generators/conan_toolchain.cmake -DCMAKE_BUILD_TYPE="${BUILD_TYPE}" ${SANITIZER_OPTION} .. -G Ninja
+        cmake \
+          -DCMAKE_TOOLCHAIN_FILE:FILEPATH=build/generators/conan_toolchain.cmake \
+          -DCMAKE_BUILD_TYPE="${BUILD_TYPE}" \
+          ${SANITIZER_OPTION} \
+          .. \
+          -G Ninja

.github/actions/get_number_of_threads/action.yml

@@ -1,5 +1,6 @@
 name: Get number of threads
 description: Determines number of threads to use on macOS and Linux
+
 inputs:
   subtract_threads:
     description: How many threads to subtract from the calculated number
@@ -9,6 +10,7 @@ outputs:
   threads_number:
     description: Number of threads to use
     value: ${{ steps.number_of_threads_export.outputs.num }}
+
 runs:
   using: composite
   steps:

.github/actions/git_common_ancestor/action.yml

@@ -1,9 +1,11 @@
 name: Git common ancestor
 description: Find the closest common commit
+
 outputs:
   commit:
     description: Hash of commit
     value: ${{ steps.find_common_ancestor.outputs.commit }}
+
 runs:
   using: composite
   steps:

.github/actions/prepare_runner/action.yml

@@ -1,9 +1,11 @@
 name: Prepare runner
 description: Install packages, set environment variables, create directories
+
 inputs:
   disable_ccache:
     description: Whether ccache should be disabled
     required: true
+
 runs:
   using: composite
   steps:

.github/actions/restore_cache/action.yml

@@ -1,5 +1,6 @@
 name: Restore cache
 description: Find and restores conan and ccache cache
+
 inputs:
   conan_dir:
     description: Path to .conan directory
@@ -28,6 +29,7 @@ outputs:
   ccache_cache_hit:
     description: True if ccache cache has been downloaded
     value: ${{ steps.ccache_cache.outputs.cache-hit }}
+
 runs:
   using: composite
   steps:

.github/actions/save_cache/action.yml

@@ -1,5 +1,6 @@
 name: Save cache
 description: Save conan and ccache cache for develop branch
+
 inputs:
   conan_dir:
     description: Path to .conan directory
@@ -29,6 +30,7 @@ inputs:
     description: Whether code coverage is on
     required: true
     default: "false"
+
 runs:
   using: composite
   steps:

.github/workflows/build.yml

@@ -68,11 +68,14 @@ jobs:
     runs-on: heavy
     container:
       image: ghcr.io/xrplf/clio-ci:latest
+
     steps:
       - uses: actions/checkout@v4
+
       - uses: actions/download-artifact@v4
         with:
           name: clio_server_Linux_Release_gcc
+
       - name: Compare Config Description
         shell: bash
         run: |

.github/workflows/build_clio_docker_image.yml

@@ -47,6 +47,7 @@ jobs:
   build_and_publish_image:
     name: Build and publish image
     runs-on: ubuntu-latest
+
     steps:
       - uses: actions/checkout@v4
 

.github/workflows/check_libxrpl.yml

@@ -80,9 +80,11 @@ jobs:
     needs: [build, run_tests]
     if: ${{ always() && contains(needs.*.result, 'failure') }}
     runs-on: ubuntu-latest
+
     permissions:
       contents: write
       issues: write
+
     steps:
       - uses: actions/checkout@v4
 

.github/workflows/check_pr_title.yml

@@ -8,6 +8,7 @@ on:
 jobs:
   check_title:
     runs-on: ubuntu-latest
+
     steps:
       - uses: ytanikin/pr-conventional-commits@8267db1bacc237419f9ed0228bb9d94e94271a1d # v1.4.1
         with:

.github/workflows/clang-tidy.yml

@@ -24,6 +24,7 @@ jobs:
     runs-on: heavy
     container:
       image: ghcr.io/xrplf/clio-ci:latest
+
     permissions:
       contents: write
       issues: write

.github/workflows/clang-tidy_on_fix_merged.yml

@@ -1,4 +1,5 @@
 name: Restart clang-tidy workflow
+
 on:
   push:
     branches: [develop]

.github/workflows/docs.yml

@@ -24,6 +24,7 @@ jobs:
     continue-on-error: true
     container:
       image: ghcr.io/xrplf/clio-ci:latest
+
     steps:
       - name: Checkout
         uses: actions/checkout@v4

.github/workflows/nightly.yml

@@ -75,9 +75,11 @@ jobs:
     needs: [build-and-test, nightly_release, build_and_publish_docker_image]
     if: ${{ always() && contains(needs.*.result, 'failure') && github.event_name != 'pull_request' }}
     runs-on: ubuntu-latest
+
     permissions:
       contents: write
       issues: write
+
     steps:
       - uses: actions/checkout@v4
 

.github/workflows/pre-commit.yml

@@ -10,6 +10,7 @@ on:
 jobs:
   run-hooks:
     runs-on: ubuntu-latest
+
     steps:
       - name: Checkout Repo ⚡️
         uses: actions/checkout@v4

.github/workflows/update_docker_ci.yml

@@ -27,6 +27,7 @@ jobs:
   build_and_push:
     name: Build and push docker image
     runs-on: [self-hosted, heavy]
+
     steps:
       - uses: actions/checkout@v4
       - uses: ./.github/actions/build_docker_image

.github/workflows/upload_coverage_report.yml

@@ -11,6 +11,7 @@ jobs:
   upload_report:
     name: Upload report
     runs-on: ubuntu-latest
+
     steps:
       - uses: actions/checkout@v4
         with: