Skip to content

Commit 7770ad1

Browse files
committed
Allow bazel.yml to access GITHUB_TOKEN
1 parent 1e97b9f commit 7770ad1

File tree

3 files changed

+6
-4
lines changed

3 files changed

+6
-4
lines changed

.github/workflows/bazel.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,8 @@ jobs:
4646
bazel:
4747
name: ${{ inputs.name }}
4848
runs-on: ${{ inputs.os }}-latest
49+
env:
50+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
4951
steps:
5052
- name: Checkout source tree
5153
uses: actions/checkout@v3
@@ -99,8 +101,6 @@ jobs:
99101
edge-version: ${{ inputs.browser-version || 'stable' }}
100102
- name: Run Bazel
101103
run: ${{ inputs.run }}
102-
env:
103-
SELENIUM_BROWSER: ${{ inputs.browser }}
104104
- name: Start SSH session
105105
if: failure() && runner.debug == '1'
106106
uses: mxschmitt/action-tmate@v3

.github/workflows/ci-javascript.yml

+3-1
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,9 @@ jobs:
4141
name: Browser Tests (${{ matrix.browser }})
4242
browser: ${{ matrix.browser }}
4343
cache-key: node
44-
run: bazel test --flaky_test_attempts 3 //javascript/node/selenium-webdriver:tests
44+
run: |
45+
export SELENIUM_BROWSER=${{ matrix.browser }}
46+
bazel test --flaky_test_attempts 3 //javascript/node/selenium-webdriver:tests
4547
4648
# As soon as this gets merged https://github.com/facebook/jest/pull/9351, we should upgrade Jest and
4749
# run bazel test javascript/grid-ui:test for these tests

.github/workflows/nightly.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,6 @@ jobs:
2323
name: Release
2424
cache-key: rb-nightly-${{ matrix.gem }}
2525
run: |
26-
export GEM_HOST_API_KEY="Bearer ${{ secrets.GITHUB_TOKEN }}"
26+
export GEM_HOST_API_KEY="Bearer $GITHUB_TOKEN"
2727
bazel run //rb:${{ matrix.gem }}-bump-nightly-version ${{ inputs.version }}
2828
bazel run //rb:${{ matrix.gem }}-release-nightly

0 commit comments

Comments
 (0)