From ad608d8d167a4f11749dfd6197eef348ff3a985c Mon Sep 17 00:00:00 2001
From: Joni Harker <jesusaurus@users.noreply.github.com>
Date: Thu, 26 Sep 2024 09:04:40 -0700
Subject: [PATCH] Fix Coveralls workflow (#24)

Fix the coverage+coveralls integration, and respect Pipfile.lock
---
 .github/workflows/test.yaml | 7 ++++++-
 .pre-commit-config.yaml     | 8 ++++----
 2 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/.github/workflows/test.yaml b/.github/workflows/test.yaml
index c1fbb10..870a741 100644
--- a/.github/workflows/test.yaml
+++ b/.github/workflows/test.yaml
@@ -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
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 60cd878..df6e9d5 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -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
@@ -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