Merge pull request #6 from yunks128/main

Enhance leaderboard generation workflow and script permissions
NASA-AMMOS · Nov 9, 2024 · 3fd1464 · 3fd1464
2 parents 75d781b + f1822fd
commit 3fd1464
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 11 deletions.
diff --git a/.github/workflows/leaderboard-generation.yml b/.github/workflows/leaderboard-generation.yml
@@ -23,10 +23,12 @@ jobs:
 
     - name: Install dependencies
       run: |
-        python -m venv .venv
-        source .venv/bin/activate
-        pip install --editable .
-    
+        python -m pip install --upgrade pip
+        python -m pip install --editable .
+        # Make the slim-leaderboard script executable and add to PATH
+        chmod +x src/jpl/slim/leaderboard.py
+        echo "${GITHUB_WORKSPACE}/src/jpl/slim" >> $GITHUB_PATH
+
     - name: Run script and prepare files
       env:
         GITHUB_TOKEN: ${{ secrets.TOKEN }}
@@ -59,8 +61,7 @@ jobs:
           Automated PR to update SLIM leaderboards
           
           Updates made to:
-          - examples/slim-fprime-leaderboard.md
-          - examples/slim-ammos-leaderboard.md
+          - examples/slim-*-leaderboard.md
           
           Generated by GitHub Actions - please review the changes.
         branch: update-slim-leaderboards

diff --git a/src/jpl/slim/leaderboard.py b/src/jpl/slim/leaderboard.py
@@ -500,11 +500,11 @@ def style_status_for_markdown(status, emoji=False):
         if args.output_format == "MARKDOWN":  # If markdown styling specified, will just print pure Markdown text not rendered
             markdown_table = textwrap.dedent("""
             # Summary Statistics
-            'YES': Full completion/success (100%)
-            'NO': No completion/failure (0%)
-            'PARTIAL': Partial completion (50%)
-            'ISSUE': Problem reported (25%)
-            'PR' (likely Pull Request): Code changes proposed (25%)
+              - 'YES': Full completion/success (100%)
+              - 'NO': No completion/failure (0%)
+              - 'PARTIAL': Partial completion (50%)
+              - 'ISSUE': Problem reported (25%)
+              - 'PR' (likely Pull Request): Code changes proposed (25%)
 
             | Category | Score (%) |
             | ------ | ----- |