Temporarily Disable Review Apps Script as deploy is NOT Working #407

dwyl · Aug 15, 2023 · 7703024 · 7703024
1 parent 634be19
commit 7703024
Showing 1 changed file with 59 additions and 59 deletions.
diff --git a/.github/workflows/pr.yml b/.github/workflows/pr.yml
@@ -61,68 +61,68 @@ jobs:
 
 
 
-  # DEPLOY THE REVIEW APP
-  # This will deploy an app to fly.io with the name 'mvp-pr-$PR_NUMBER' (check `review-apps.sh` script).
-  review_app:
-
-    # Only run when it's not a dependabot PR
-    if: github.event.pull_request.user.login != 'dependabot[bot]'
-
-    name: Review App Job
-    runs-on: ubuntu-latest
-    # needs: [build]
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@v3
-
-      - name: Install flyctl
-        uses: superfly/flyctl-actions/setup-flyctl@master
-
-      - name: Set up Elixir
-        uses: erlef/setup-beam@v1
-        with:
-          otp-version: 24.3.4
-          elixir-version: 1.14.1
-
-      - name: Run Review App Script
-        run: ./.github/scripts/review-apps.sh
-        env:
-          ENCRYPTION_KEYS: ${{ secrets. ENCRYPTION_KEYS }}
-          AUTH_API_KEY: ${{ secrets.FLY_AUTH_API_KEY }}
-          APP_NAME: ${{ format('mvp-pr-{0}', github.event.number) }}
-          EVENT_ACTION: ${{ github.event.action }}
-          FLY_API_TOKEN: ${{ secrets.FLY_API_TOKEN }}
-          FLY_ORG: dwyl-mvp
-          FLY_REGION: lhr
-          FLY_POSTGRES_NAME: mvp-db
+  # # DEPLOY THE REVIEW APP
+  # # This will deploy an app to fly.io with the name 'mvp-pr-$PR_NUMBER' (check `review-apps.sh` script).
+  # review_app:
+
+  #   # Only run when it's not a dependabot PR
+  #   if: github.event.pull_request.user.login != 'dependabot[bot]'
+
+  #   name: Review App Job
+  #   runs-on: ubuntu-latest
+  #   # needs: [build]
+  #   steps:
+  #     - name: Checkout repository
+  #       uses: actions/checkout@v3
+
+  #     - name: Install flyctl
+  #       uses: superfly/flyctl-actions/setup-flyctl@master
+
+  #     - name: Set up Elixir
+  #       uses: erlef/setup-beam@v1
+  #       with:
+  #         otp-version: 24.3.4
+  #         elixir-version: 1.14.1
+
+  #     - name: Run Review App Script
+  #       run: ./.github/scripts/review-apps.sh
+  #       env:
+  #         ENCRYPTION_KEYS: ${{ secrets. ENCRYPTION_KEYS }}
+  #         AUTH_API_KEY: ${{ secrets.FLY_AUTH_API_KEY }}
+  #         APP_NAME: ${{ format('mvp-pr-{0}', github.event.number) }}
+  #         EVENT_ACTION: ${{ github.event.action }}
+  #         FLY_API_TOKEN: ${{ secrets.FLY_API_TOKEN }}
+  #         FLY_ORG: dwyl-mvp
+  #         FLY_REGION: lhr
+  #         FLY_POSTGRES_NAME: mvp-db
 
 
   # API DEFINITION TESTING - https://docs.hoppscotch.io/cli 
   # NOTE: The tests are executed sequentially in each folder
   # but all the sequence of folders are run in reverse order of what's displayed in the GUI.
   # We've made use of env variables to make it work. Take this into account if you want to add more requests.
-  api_definition:
-
-    # Only run when not closed
-    if: github.event.pull_request.action != 'closed'
-
-    name: API Definition Tests
-    runs-on: ubuntu-latest
-    needs: [review_app]
-    steps:
-    - uses: actions/checkout@v3
-
-    # After the app is deployed, we need to change the host
-    # in the `lib/api/fly_dev.json` so the API tests work and target the deployed app.
-    - name: Update lib/api/fly_dev.json description
-      uses: jossef/action-set-json-field@v2.1
-      with:
-        file: ./lib/api/fly_dev.json
-        field: host
-        value: ${{ format('https://mvp-pr-{0}.fly.dev', github.event.number) }}
-
-    - name: Install Hoppscotch CLI
-      run: npm i -g @hoppscotch/cli@0.3.0
-
-    - name: Running server and Hoppscotch Tests
-      run: hopp test -e ./lib/api/fly_dev.json ./lib/api/MVP.json
+  # api_definition:
+
+  #   # Only run when not closed
+  #   if: github.event.pull_request.action != 'closed'
+
+  #   name: API Definition Tests
+  #   runs-on: ubuntu-latest
+  #   needs: [review_app]
+  #   steps:
+  #   - uses: actions/checkout@v2
+
+  #   # After the app is deployed, we need to change the host
+  #   # in the `lib/api/fly_dev.json` so the API tests work and target the deployed app.
+  #   - name: Update lib/api/fly_dev.json description
+  #     uses: jossef/action-set-json-field@v2.1
+  #     with:
+  #       file: ./lib/api/fly_dev.json
+  #       field: host
+  #       value: ${{ format('https://mvp-pr-{0}.fly.dev', github.event.number) }}
+
+  #   - name: Install Hoppscotch CLI
+  #     run: npm i -g @hoppscotch/cli@0.3.0
+
+  #   - name: Running server and Hoppscotch Tests
+  #     run: hopp test -e ./lib/api/fly_dev.json ./lib/api/MVP.json