Skip to content

Commit

Permalink
Fix Coveralls workflow (#24)
Browse files Browse the repository at this point in the history
Fix the coverage+coveralls integration, and respect Pipfile.lock
  • Loading branch information
jesusaurus authored Sep 26, 2024
1 parent 6705638 commit ad608d8
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 5 deletions.
7 changes: 6 additions & 1 deletion .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -25,10 +25,15 @@ jobs:
with:
python-version: 3.8
- run: pip install -U pipenv
- run: pipenv install --dev
- run: pipenv sync --dev
- run: pipenv run coverage run -m pytest tests/ -svv
- run: pipenv run coverage lcov
- name: upload coverage to coveralls
uses: coverallsapp/github-action@v2
with:
file: coverage.lcov
fail-on-error: false


sam-build-and-lint:
runs-on: ubuntu-latest
Expand Down
8 changes: 4 additions & 4 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
repos:
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v4.1.0
rev: v4.6.0
hooks:
- id: end-of-file-fixer
language_version: python3
Expand All @@ -9,18 +9,18 @@ repos:
- id: check-ast
language_version: python3
- repo: https://github.com/adrienverge/yamllint
rev: v1.26.3
rev: v1.35.1
hooks:
- id: yamllint
language_version: python3
- repo: https://github.com/awslabs/cfn-python-lint
rev: v0.58.0
rev: v1.15.1
hooks:
- id: cfn-python-lint
files: template\.(json|yml|yaml)$
language_version: python3
- repo: https://github.com/Lucas-C/pre-commit-hooks
rev: v1.1.11
rev: v1.5.5
hooks:
- id: remove-tabs
language_version: python3

0 comments on commit ad608d8

Please sign in to comment.