From 956f6610ec6fe08f0bfe000b3f722caa273e104d Mon Sep 17 00:00:00 2001
From: Nick Pope <nick@nickpope.me.uk>
Date: Wed, 1 May 2024 09:22:04 +0100
Subject: [PATCH] Removed obsolete logic from screenshots workflow.

The id was used when we attempted to update comments in an early
version of the feature. Also removed the job summary as it
doesn't have the value that it did in the original version of
the feature.
---
 .github/workflows/screenshots.yml | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/.github/workflows/screenshots.yml b/.github/workflows/screenshots.yml
index c85a258949..7714f99710 100644
--- a/.github/workflows/screenshots.yml
+++ b/.github/workflows/screenshots.yml
@@ -30,14 +30,10 @@ jobs:
       - name: Install and upgrade packaging tools
         run: python -m pip install --upgrade pip setuptools wheel
       - run: python -m pip install -r tests/requirements/py3.txt -e .
+
       - name: Run Selenium tests with screenshots
-        id: generate-screenshots
         working-directory: ./tests/
-        run: |
-          python -Wall runtests.py --verbosity 2 --noinput --selenium=chrome --headless --screenshots --settings=test_sqlite --parallel 2
-          echo "date=$(date)" >> $GITHUB_OUTPUT
-          echo "🖼️ **Screenshots created**" >> $GITHUB_STEP_SUMMARY
-          echo "Generated screenshots for ${{ github.event.pull_request.head.sha }} at $(date)" >> $GITHUB_STEP_SUMMARY
+        run: python -Wall runtests.py --verbosity=2 --noinput --selenium=chrome --headless --screenshots --settings=test_sqlite --parallel=2
 
       - name: Upload screenshots
         uses: actions/upload-artifact@v4