Refactor workflows to run tests on PRs (#89)

Author: jakubno

.github/workflows/charts_tests.yml

@@ -11,7 +11,7 @@ jobs:
     defaults:
       run:
         working-directory: ./chart_data_extractor
-    name: Build and test Chart Data Extractor
+    name: Chart Data Extractor - Build and test
     runs-on: ubuntu-22.04
     steps:
       - name: Checkout repository

.github/workflows/js_tests.yml

@@ -14,7 +14,7 @@ jobs:
     defaults:
       run:
         working-directory: ./js
-    name: Build and test SDK
+    name: JS SDK - Build and test
     runs-on: ubuntu-22.04
     steps:
       - name: Checkout repository

.github/workflows/pull_request.yml

@@ -0,0 +1,26 @@
+name: Pull Request
+
+permissions:
+  contents: read
+  id-token: write
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
+on:
+  pull_request:
+    branches:
+      - main
+
+jobs:
+  js-sdk:
+    uses: ./.github/workflows/js_tests.yml
+    secrets:
+      E2B_API_KEY: ${{ secrets.E2B_API_KEY }}
+  python-sdk:
+    uses: ./.github/workflows/python_tests.yml
+    secrets:
+      E2B_API_KEY: ${{ secrets.E2B_API_KEY }}
+  charts-tests:
+    uses: ./.github/workflows/charts_tests.yml

.github/workflows/python_tests.yml

@@ -14,7 +14,7 @@ jobs:
     defaults:
       run:
         working-directory: ./python
-    name: Build and test SDK
+    name: Python SDK - Build and test
     runs-on: ubuntu-22.04
     steps:
       - name: Checkout repository

.github/workflows/release.yml

@@ -108,15 +108,9 @@ jobs:
           IS_RELEASE=$(./.github/scripts/is_release_for_package.sh "@e2b/code-interpreter-template")
           echo "release=$IS_RELEASE" >> "$GITHUB_OUTPUT"
 
-  charts-tests:
-    name: Charts tests
-    needs: [changes]
-    if: needs.changes.outputs.charts == 'true'
-    uses: ./.github/workflows/charts_tests.yml
 
   charts-release:
     name: Charts release
-    needs: [charts-tests]
     if: needs.changes.outputs.charts == 'true'
     runs-on: ubuntu-latest
     outputs:
@@ -243,8 +237,8 @@ jobs:
     needs: [changes, build-template]
     if: always() && 
       !contains(needs.*.result, 'failure') &&
-      !contains(needs.*.result, 'cancelled') &&
-      (needs.changes.outputs.python == 'true' || needs.changes.outputs.template == 'true')
+      !contains(needs.*.result, 'cancelled') && 
+      needs.changes.outputs.template == 'true'
     uses: ./.github/workflows/python_tests.yml
     secrets: inherit
 
@@ -254,7 +248,7 @@ jobs:
     if: always() && 
       !contains(needs.*.result, 'failure') &&
       !contains(needs.*.result, 'cancelled') &&
-      (needs.changes.outputs.js == 'true' || needs.changes.outputs.template == 'true')
+      needs.changes.outputs.template == 'true'
     uses: ./.github/workflows/js_tests.yml
     secrets: inherit