updated CI

Author: mtelek

.github/workflows/build.yml

@@ -11,7 +11,7 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-    - uses: actions/checkout@v3
+    - uses: actions/checkout@v4
 
     - name: Install dependencies
       run: |
@@ -25,28 +25,29 @@ jobs:
 
     - name: Basic map validation
       run: |
-        ./cub3D maps/invalid_map.cub 2>&1 | grep "Error" && exit 1 || echo "Map validation working"
+        ./cub3D maps/invalid_map.cub 2>&1 | grep "Error" || (echo "Map validation failed" && exit 1)
         ./cub3D maps/valid_map.cub 2>&1 | grep "Error" && exit 1 || echo "Valid map accepted"
 
-    - name: Run with Valgrind (leak check)
+    - name: Run with Valgrind
       run: |
         valgrind --leak-check=full --show-leak-kinds=all --error-exitcode=1 \
         --track-origins=yes --log-file=valgrind.log \
-        ./cub3D maps/valid_map.cub
+        ./cub3D maps/valid_map.cub || true
         cat valgrind.log
 
     - name: Output verification
       run: |
         timeout 5 ./cub3D maps/valid_map.cub > output.log 2>&1 &
         sleep 3
         grep "Initialization complete" output.log || (echo "Startup failed" && exit 1)
-        killall cub3D
+        killall cub3D || true
 
-    - name: Save artifacts on failure
-      if: failure()
-      uses: actions/upload-artifact@v3
+    - name: Upload debug logs
+      if: always() 
+      uses: actions/upload-artifact@v4
       with:
         name: debug-logs
         path: |
           output.log
-          valgrind.log
+          valgrind.log
+        retention-days: 1