Allow bazel.yml to access GITHUB_TOKEN

p0deje · p0deje · commit 7770ad12977f · 2023-01-14T10:59:01.000-08:00
diff --git a/.github/workflows/bazel.yml b/.github/workflows/bazel.yml
@@ -46,6 +46,8 @@ jobs:
   bazel:
     name: ${{ inputs.name }}
     runs-on: ${{ inputs.os }}-latest
+    env:
+      GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
     steps:
       - name: Checkout source tree
         uses: actions/checkout@v3
@@ -99,8 +101,6 @@ jobs:
           edge-version: ${{ inputs.browser-version || 'stable' }}
       - name: Run Bazel
         run: ${{ inputs.run }}
-        env:
-          SELENIUM_BROWSER: ${{ inputs.browser }}
       - name: Start SSH session
         if: failure() && runner.debug == '1'
         uses: mxschmitt/action-tmate@v3
diff --git a/.github/workflows/ci-javascript.yml b/.github/workflows/ci-javascript.yml
@@ -41,7 +41,9 @@ jobs:
       name: Browser Tests  (${{ matrix.browser }})
       browser: ${{ matrix.browser }}
       cache-key: node
-      run: bazel test --flaky_test_attempts 3 //javascript/node/selenium-webdriver:tests
+      run: |
+        export SELENIUM_BROWSER=${{ matrix.browser }}
+        bazel test --flaky_test_attempts 3 //javascript/node/selenium-webdriver:tests
 
   # As soon as this gets merged https://github.com/facebook/jest/pull/9351, we should upgrade Jest and
   # run bazel test javascript/grid-ui:test for these tests
diff --git a/.github/workflows/nightly.yml b/.github/workflows/nightly.yml
@@ -23,6 +23,6 @@ jobs:
       name: Release
       cache-key: rb-nightly-${{ matrix.gem }}
       run: |
-        export GEM_HOST_API_KEY="Bearer ${{ secrets.GITHUB_TOKEN }}"
+        export GEM_HOST_API_KEY="Bearer $GITHUB_TOKEN"
         bazel run //rb:${{ matrix.gem }}-bump-nightly-version ${{ inputs.version }}
         bazel run //rb:${{ matrix.gem }}-release-nightly
