Enhance contributors workflow with sorting and lazy loading

Sort contributors list and add lazy loading to images

Author: steam-bell-92

.github/workflows/contributors.yml

@@ -3,17 +3,18 @@ name: contributor automation
 on:
   schedule: # runs daily at midnight
     - cron: '0 0 * * *'     
-  push:
+  pull_request:
+    types: [closed]
     branches:
-      - main          # also updates when new pr is merged into the main
-  workflow_dispatch:
+      - main
 
       # allows manual trigger
 permissions:
   contents: write
 
 jobs:
   update-readme:
+    if: github.event_name != 'pull_request' || github.event.pull_request.merged == true
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
@@ -35,14 +36,17 @@ jobs:
             PAGE=$((PAGE + 1))
           done
           # Filters for actual merged PRs and unique users
-          jq '[.[] | select(.merged_at != null and .user != null and .user.type != "Bot") | .user.login] | unique' final_list.json > contributors.json
-
+          jq '[.[] 
+            | select(.merged_at != null and .user != null and .user.type != "Bot") 
+            | .user.login] 
+            | unique 
+            | sort' final_list.json > contributors.json
       - name: Build Gallery HTML
         run: |
           set -euo pipefail
           GALLERY_HTML=""
           for USERNAME in $(jq -r '.[]' contributors.json); do
-            GALLERY_HTML="${GALLERY_HTML}<a href=\"https://github.com/${USERNAME}\"><img src=\"https://github.com/${USERNAME}.png\" width=\"50px\" style=\"border-radius:50%;margin:5px;\" alt=\"${USERNAME}\" /></a>"
+            GALLERY_HTML="${GALLERY_HTML}<a href=\"https://github.com/${USERNAME}\"><img src=\"https://github.com/${USERNAME}.png\" width=\"50px\" loading=\"lazy\" title=\"${USERNAME}\" style=\"border-radius:50%;margin:5px;\" alt=\"${USERNAME}\" /></a>"
           done
           echo "$GALLERY_HTML" > gallery_fragment.txt