Update main.yml step order (#77)

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

- **Chores**
- Improved artifact management by adding a new step for uploading
binlogs after test execution.
- Removed redundant upload step for binlogs and adjusted package upload
to run unconditionally.
- Streamlined workflow configuration while maintaining existing
structure and functionality.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Signed-off-by: Richard Murillo <[email protected]>

Author: rjmurillo

Diff for: .github/workflows/main.yml

@@ -84,6 +84,14 @@ jobs:
     - name: Test
       run: dotnet test --no-build --configuration Release --settings ./build/targets/tests/test.runsettings
 
+    - name: Upload binlogs
+      uses: actions/upload-artifact@v4
+      if: success() || failure()
+      with:
+        name: binlogs-${{ matrix.os }}
+        path: ./artifacts/logs
+        if-no-files-found: error
+
     - name: Upload *.received.* files
       uses: actions/upload-artifact@v4
       if: failure()
@@ -125,17 +133,8 @@ jobs:
         project-token: ${{ secrets.CODACY_PROJECT_TOKEN }}
         coverage-reports: ${{ github.workspace }}/artifacts/TestResults/coverage/Cobertura.xml
 
-    - name: Upload binlogs
-      uses: actions/upload-artifact@v4
-      if: success() || failure()
-      with:
-        name: binlogs-${{ matrix.os }}
-        path: ./artifacts/logs
-        if-no-files-found: error
-
     - name: Upload packages
       uses: actions/upload-artifact@v4
-      if: success()
       with:
         name: packages-${{ matrix.os }}
         path: |