minor ci improvements

anoma · Aug 21, 2024 · 4043eee · 4043eee
1 parent 78323a9
commit 4043eee
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 6 deletions.
diff --git a/.github/workflows/pre-genesis-transactions-checks.yml b/.github/workflows/pre-genesis-transactions-checks.yml
@@ -55,7 +55,8 @@ jobs:
       - name: Install dependencies
         run: poetry install --no-interaction
         if: steps.cached-poetry-dependencies.outputs.cache-hit != 'true'
-      - run: |
+      - name: Check PR submission correctness
+        run: |
           source .venv/bin/activate
           poetry run python scripts/validate-pr.py
         env:
@@ -96,7 +97,8 @@ jobs:
       - name: Install dependencies
         run: poetry install --no-interaction
         if: steps.cached-poetry-dependencies.outputs.cache-hit != 'true'
-      - run: |
+      - name: Check PR submission correctness
+        run: |
           source .venv/bin/activate
           poetry run python scripts/validate-merge.py
 
@@ -135,7 +137,8 @@ jobs:
       - name: Install dependencies
         run: poetry install --no-interaction
         if: steps.cached-poetry-dependencies.outputs.cache-hit != 'true'
-      - run: |
+      - name: Rebuild README.md
+        run: |
           source .venv/bin/activate
           poetry run python scripts/rebuild.py
       - uses: stefanzweifel/git-auto-commit-action@v5

diff --git a/scripts/validate-pr.py b/scripts/validate-pr.py
@@ -14,9 +14,9 @@ def read_env():
     can_apply_for_bonds = os.environ.get('CAN_ADD_BONDS', True).lower() in ('true', '1', 't')
     can_apply_for_accounts = os.environ.get('CAN_ADD_ACCOUNTS', True).lower() in ('true', '1', 't')
 
-    print("Can add validators: {}", can_apply_for_validators)
-    print("Can add bonds: {}", can_apply_for_bonds)
-    print("Can add accounts: {}", can_apply_for_accounts)
+    print("Can add validators: {}".format(can_apply_for_validators))
+    print("Can add bonds: {}".format(can_apply_for_bonds))
+    print("Can add accounts: {}".format(can_apply_for_accounts))
 
     return can_apply_for_validators, can_apply_for_bonds, can_apply_for_accounts