From ca98584fd0beaa5b03a9a0708309fb5f21ba9a46 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Kevin=20L=C3=B3pez=20Brante?= <kevin@kddlb.cl>
Date: Sat, 10 Oct 2020 20:45:39 -0300
Subject: [PATCH] Modified GHA to not upload to S3 when PR is built

---
 .github/workflows/cmake-lx.yml | 3 ++-
 .github/workflows/vs-win.yml   | 7 ++-----
 2 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/.github/workflows/cmake-lx.yml b/.github/workflows/cmake-lx.yml
index 58f94352..64934536 100644
--- a/.github/workflows/cmake-lx.yml
+++ b/.github/workflows/cmake-lx.yml
@@ -1,6 +1,6 @@
 name: CMake Linux
 
-on: [push]
+on: [push, pull_request]
 
 env:
   # Customize the CMake build type here (Release, Debug, RelWithDebInfo, etc.)
@@ -59,6 +59,7 @@ jobs:
           name: vgmstream-audacious
 
     - name: Upload Artifacts to S3
+      if: github.event_name != 'pull_request'
       working-directory: ${{runner.workspace}}/build
       shell: bash
       env:
diff --git a/.github/workflows/vs-win.yml b/.github/workflows/vs-win.yml
index 251ab08b..a7bc1d19 100644
--- a/.github/workflows/vs-win.yml
+++ b/.github/workflows/vs-win.yml
@@ -4,11 +4,7 @@ name: Windows VS 2017
 
 # Controls when the action will run. Triggers the workflow on push or pull request
 # events but only for the master branch
-on:
-  push:
-    branches: [ master ]
-  pull_request:
-    branches: [ master ]
+on: [push, pull_request]
 
 # A workflow run is made up of one or more jobs that can run sequentially or in parallel
 jobs:
@@ -105,6 +101,7 @@ jobs:
           path: ${{github.workspace}}\tmp\cli-p
           
       - name: Upload Artifacts to S3
+        if: github.event_name != 'pull_request'
         working-directory: ${{github.workspace}}
         shell: powershell
         env: