Skip to content

Commit

Permalink
Merge pull request #285 from stronk7/avoid_node16_deprecation_message
Browse files Browse the repository at this point in the history
GHA is deprecating Node 16 actions, let's bump them
  • Loading branch information
stronk7 authored Nov 15, 2023
2 parents aad0296 + 8abcf23 commit 4c4b34e
Showing 1 changed file with 14 additions and 5 deletions.
19 changes: 14 additions & 5 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,12 @@ jobs:
runs-on: ubuntu-22.04
steps:
- name: Checking out moodle-local_ci
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
path: local_ci

- name: Checking out moodle
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
repository: moodle/moodle
fetch-depth: 0 # We need a complete clone, because we use various commits / tags.
Expand Down Expand Up @@ -50,7 +50,7 @@ jobs:
coverage: none

- name: Configuring node & npm
uses: actions/setup-node@v3
uses: actions/setup-node@v4
with:
node-version-file: 'moodle/.nvmrc'

Expand All @@ -61,9 +61,15 @@ jobs:
npm install
- name: Setup Bats
uses: amdilabs/setup-bats@v1
uses: brokenpip3/setup-bats-libs@1.5.2
with:
bats-version: 1.9.0
bats-version: 1.10.0
# We need only support and assert libraries, and installed locally for caching to work.
# See https://github.com/brokenpip3/setup-bats-libs/issues/18
support-path: "${{ github.workspace }}/.bats/bats-support"
assert-path: "${{ github.workspace }}/.bats/bats-assert"
detik-install: false
file-install: false

- name: Run Tests
working-directory: local_ci # Tests have to start from here, because some use $PWD to detect the local_ci base.
Expand All @@ -76,6 +82,9 @@ jobs:
LOCAL_CI_TESTS_DBHOST: 127.0.0.1
LOCAL_CI_TESTS_DBUSER: root
LOCAL_CI_TESTS_DBPASS: test
# We need to specify where the libraries are, installed locally for caching to work.
# See https://github.com/brokenpip3/setup-bats-libs/issues/18
BATS_LIB_PATH: "${{ github.workspace }}/.bats"
run: |
mkdir -p ${LOCAL_CI_TESTS_CACHEDIR}
bats --timing tests/${{ matrix.test }}

0 comments on commit 4c4b34e

Please sign in to comment.