Merge pull request #124 from LedgerHQ/fbe/fix_disable_legacy_snapshot_cleaning_for_now

Fbe/fix disable legacy snapshot cleaning for now

Author: fbeutin-ledger

.github/workflows/_open_pr_with_new_snapshots.yml

@@ -48,10 +48,6 @@ jobs:
           # instead of provided ones
           persist-credentials: false
 
-      - name: Clear legacy snapshots
-        run: |
-          rm -rf ${{ inputs.snapshots_directory }}/*
-
       - name: Download new snapshots
         uses: actions/download-artifact@v4
         with:
@@ -61,6 +57,7 @@ jobs:
       - name: Check if update is needed
         id: check-changes
         run: |
+          git status
           if [ -n "$(git status --porcelain)" ]; then
               echo "Changes detected"
               echo "changes_detected=true" >> "$GITHUB_OUTPUT"

.github/workflows/reusable_ragger_tests.yml

@@ -179,6 +179,11 @@ jobs:
             ./${{ needs.call_get_app_metadata.outputs.pytest_directory }}/setup_script.sh
           fi
 
+      - name: Clear legacy snapshots
+        if: ${{ inputs.regenerate_snapshots == true }}
+        run: |
+          rm -rf ${{ needs.call_get_app_metadata.outputs.pytest_directory }}/snapshots
+
       - name: Run test
         env:
           CTEST_OUTPUT_ON_FAILURE: 1