Skip to content

Commit

Permalink
Send coverage to Coveralls
Browse files Browse the repository at this point in the history
When updating CI we missed this step. We are not sure anyone actually
uses the Coveralls, but we where sending it so should continue to.

To get the report, we copy out of the container and if the step fails,
it will not effect the overall checks.
  • Loading branch information
mec committed Nov 21, 2024
1 parent 4456bc3 commit 4670f2b
Showing 1 changed file with 11 additions and 1 deletion.
12 changes: 11 additions & 1 deletion .github/workflows/ci_checks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,10 +51,20 @@ jobs:
name: Build and cache
uses: ./.github/workflows/build_and_cache
-
name: Run Rspec and Simplebcov
name: Run Rspec and Simplecov
run: |
docker compose -p app_test -f docker-compose.ci.yml \
run --name app_test test /bin/bash -c "bin/rspec --format=documentation"
-
name: Copy coverage report from container
run: mkdir coverage
run: docker cp app_test:/srv/app/coverage/lcov.info coverage/lcov.info
-
name: Shutdown containers
run: docker compose -p app_test down && docker compose -p app_test rm
-
name: Send coverage report to Coveralls
uses: coverallsapp/github-action@v2
with:
file: ./coverage/lcov.info
fail-on-error: false

0 comments on commit 4670f2b

Please sign in to comment.