Skip to content

Commit

Permalink
fix: GitHub ci
Browse files Browse the repository at this point in the history
  • Loading branch information
abeidahmed committed Feb 7, 2025
1 parent d71c4bc commit ac7210a
Showing 1 changed file with 8 additions and 26 deletions.
34 changes: 8 additions & 26 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,8 @@ jobs:
lint_js:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
with:
node-version-file: ".tool-versions"
cache: "yarn"
Expand All @@ -22,7 +22,7 @@ jobs:
lint_ruby:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: ruby/setup-ruby@v1
with:
bundler: default
Expand All @@ -47,8 +47,8 @@ jobs:
BUNDLE_GEMFILE: ${{ github.workspace }}/gemfiles/${{ matrix.gemfile }}.gemfile
BUNDLE_PATH_RELATIVE_TO_CWD: true
steps:
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
with:
node-version-file: ".tool-versions"
cache: "yarn"
Expand All @@ -63,13 +63,14 @@ jobs:
rubygems: latest
- name: Run test
run: bundle exec rake test:system
- uses: actions/upload-artifact@v3
- uses: actions/upload-artifact@v4
if: ${{ failure() }}
with:
path: |
/home/runner/work/view_components/view_components/demo/tmp/capybara/
if-no-files-found: ignore
retention-days: 7
overwrite: true

test_components:
needs:
Expand All @@ -87,7 +88,7 @@ jobs:
BUNDLE_GEMFILE: ${{ github.workspace }}/gemfiles/${{ matrix.gemfile }}.gemfile
BUNDLE_PATH_RELATIVE_TO_CWD: true
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: ruby/setup-ruby@v1
with:
ruby-version: ${{ matrix.ruby }}
Expand All @@ -96,22 +97,3 @@ jobs:
rubygems: latest
- name: Run test
run: bundle exec rake test:components

test_js:
needs:
- lint_js
- lint_ruby
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
with:
node-version-file: ".tool-versions"
cache: "yarn"
- name: Install playwright
run: npx playwright install --with-deps
- name: Run test
run: |
yarn install --frozen-lockfile
yarn build:js
yarn test

0 comments on commit ac7210a

Please sign in to comment.