Refactor retention days logic in main.yml to use a dedicated step for better clarity and maintainability.

Author: kingbuzzman

.github/workflows/main.yml

@@ -179,20 +179,29 @@ jobs:
           coverage html
           coverage report --format=markdown >> $GITHUB_STEP_SUMMARY
 
+      - name: Determine retention days
+        id: determine-retention-days
+        run: |
+          if [ "${{ github.ref }}" = "refs/heads/main" ] || [[ "${{ github.ref }}" == refs/tags/* ]]; then
+            echo "retention_days=90" >> $GITHUB_OUTPUT
+          else
+            echo "retention_days=3" >> $GITHUB_OUTPUT
+          fi
+
       - name: Upload combined .coverage file
         uses: actions/upload-artifact@v4
         with:
           name: coverage-file
           path: .coverage
-          retention-days: ${{ (github.ref == 'refs/heads/main' || startsWith(github.ref, 'refs/tags/')) && 90 || 3 }}
+          retention-days: ${{ steps.determine-retention-days.outputs.retention_days }}
           include-hidden-files: true
 
       - name: Upload HTML coverage report
         uses: actions/upload-artifact@v4
         with:
           name: coverage-html-report
           path: htmlcov
-          retention-days: ${{ (github.ref == 'refs/heads/main' || startsWith(github.ref, 'refs/tags/')) && 90 || 3 }}
+          retention-days: ${{ steps.determine-retention-days.outputs.retention_days }}
 
       - name: Delete temporary coverage artifacts from run
         uses: geekyeggo/delete-artifact@v5