disable upload-artifact until can get a more satifactory solution

.github/workflows/cypress_tests.yml

@@ -104,12 +104,13 @@ jobs:
           npm install .
           ./cypress/cypress_start.sh
 
-      - name: Upload Screenshot Artifact
-        if: failure()
-        uses: actions/[email protected]
-        with:
-          name: cypress-screenshots
-          path: cypress/screenshots/
+      # Note - V 3 of upload-artifact is deprecated but V 4 disallows uploading artifact of the same name. Commented for now but needs rework
+      #- name: Upload Screenshot Artifact
+      #  if: failure()
+      #  uses: actions/[email protected]
+      #  with:
+      #    name: cypress-screenshots
+      #    path: cypress/screenshots/
 
       - name: Configure AWS Credentials
         if: always()

.github/workflows/phpunit.yml

@@ -57,11 +57,12 @@ jobs:
         run: XDEBUG_MODE=coverage php ./vendor/bin/phpunit --coverage-html ./coverage-html --coverage-xml ./coverage-xml
       - name: Check coverage
         run: php ../scripts/pipeline/php_coverage/check_coverage.php ./coverage-xml/index.xml ${{ matrix.scan.minCoverage }}
-      - name: Upload Coverage Artifacts
-        uses: actions/[email protected]
-        with:
-          name: coverage-html
-          path: ${{ matrix.scan.path }}/coverage-html/
+      # Note - V 3 of upload-artifact is deprecated but V 4 disallows uploading artifact of the same name. Commented for now but needs rework
+      #- name: Upload Coverage Artifacts
+      #  uses: actions/[email protected]
+      #  with:
+      #    name: coverage-html
+      #    path: ${{ matrix.scan.path }}/coverage-html/
   phpunit_all_services:
     name: phpunit
     runs-on: ubuntu-latest
@@ -103,8 +104,9 @@ jobs:
         run: XDEBUG_MODE=coverage php ./vendor/bin/phpunit --coverage-html ./coverage-html --coverage-xml ./coverage-xml
       - name: Check coverage
         run: php ../scripts/pipeline/php_coverage/check_coverage.php ./coverage-xml/index.xml ${{ matrix.scan.minCoverage }}
-      - name: Upload Coverage Artifacts
-        uses: actions/[email protected]
-        with:
-          name: coverage-html
-          path: ${{ matrix.scan.path }}/coverage-html/
+      # Note - V 3 of upload-artifact is deprecated but V 4 disallows uploading artifact of the same name. Commented for now but needs rework
+      #- name: Upload Coverage Artifacts
+      #  uses: actions/[email protected]
+      #  with:
+      #    name: coverage-html
+      #    path: ${{ matrix.scan.path }}/coverage-html/