Merge pull request #15 from jesusaurus/coveralls

Fix Coveralls workflow
Sage-Bionetworks-IT · Sep 26, 2024 · c979667 · c979667
2 parents b2349d3 + f849e86
commit c979667
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/.github/workflows/test.yaml b/.github/workflows/test.yaml
@@ -25,10 +25,14 @@ 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

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -1,20 +1,20 @@
 repos:
 -   repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.3.0
+    rev: v4.6.0
     hooks:
     -   id: end-of-file-fixer
     -   id: trailing-whitespace
     -   id: check-ast
 -   repo: https://github.com/adrienverge/yamllint
-    rev: v1.27.1
+    rev: v1.35.1
     hooks:
     -   id: yamllint
 -   repo: https://github.com/awslabs/cfn-python-lint
-    rev: v0.61.3
+    rev: v1.15.1
     hooks:
     -   id: cfn-python-lint
         files: template\.(json|yml|yaml)$
 -   repo: https://github.com/Lucas-C/pre-commit-hooks
-    rev: v1.3.0
+    rev: v1.5.5
     hooks:
     -   id: remove-tabs