Skip to content

Commit

Permalink
Merge pull request #5119 from dodona-edu/chore/dont-build-twice
Browse files Browse the repository at this point in the history
Only build js once during rails test
  • Loading branch information
jorg-vr authored Nov 8, 2023
2 parents f8052d3 + 6741368 commit df937f2
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,9 +39,6 @@ jobs:
with:
node-version: 16.x
cache: yarn
- name: Install JS dependencies
run: |
yarn install
- name: Run tests
env:
CI: true
Expand All @@ -51,8 +48,6 @@ jobs:
git config --global user.name "Dodona"
sudo sysctl fs.inotify.max_user_watches=524288
sudo sysctl -p
bin/rake css:build
yarn build:js
bundle exec rails test
- name: Upload coverage to Codecov
uses: codecov/codecov-action@v3
Expand Down Expand Up @@ -114,9 +109,6 @@ jobs:
cache: yarn
- name: Setup chromium-chromedriver
uses: nanasess/setup-chromedriver@master
- name: Install JS dependencies
run: |
yarn install
- name: Show installed chrome version
run: |
google-chrome --version
Expand Down

0 comments on commit df937f2

Please sign in to comment.