Skip to content

Commit

Permalink
Use install-chromedriver option for chromedriver installation (#10)
Browse files Browse the repository at this point in the history
* Use install-chromedriver option

* Bump versions in CI
  • Loading branch information
ryanking13 authored Sep 17, 2024
1 parent e3edcb8 commit 8739164
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 9 deletions.
10 changes: 5 additions & 5 deletions .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ jobs:
os: "ubuntu-latest",
runner: "selenium",
browser: "node",
browser-version: "18",
browser-version: "22",
},
{
os: "macos-latest",
Expand All @@ -56,7 +56,7 @@ jobs:
- uses: actions/checkout@v3
- uses: actions/setup-python@v4
with:
python-version: "3.11"
python-version: "3.12"
- uses: ./install-browser
with:
runner: ${{ matrix.test-config.runner }}
Expand All @@ -74,13 +74,13 @@ jobs:
strategy:
fail-fast: false
matrix:
version: [0.21.3, 0.22.1, 0.23.2]
version: [0.23.2, 0.24.0, 0.25.0, 0.26.0]
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/setup-python@v4
with:
python-version: "3.11"
python-version: "3.12"
- uses: ./download-pyodide
with:
version: ${{ matrix.version }}
Expand All @@ -98,5 +98,5 @@ jobs:
- uses: actions/checkout@v3
- uses: actions/setup-python@v4
with:
python-version: "3.11"
python-version: "3.12"
- uses: pre-commit/action@v3.0.0
5 changes: 1 addition & 4 deletions install-browser/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -76,10 +76,7 @@ runs:
if: ${{ inputs.runner == 'selenium' && contains(inputs.browser, 'chrome') }}
with:
chrome-version: ${{ inputs.browser-version }}

- name: Install chromedriver
if: ${{ inputs.runner == 'selenium' && contains(inputs.browser, 'chrome') }}
uses: nanasess/setup-chromedriver@v2
install-chromedriver: true

- name: Enable Safari Driver
shell: bash -l {0}
Expand Down

0 comments on commit 8739164

Please sign in to comment.