chore: update Makefile and CONTRIBUTING.md

Author: Daniel-Brai

.github/workflows/ci.yml

@@ -32,15 +32,26 @@ jobs:
         with:
           python-version-file: ".python-version"
 
-      - name: Build the project
+      - name: Build Docker image
         run: |
-          docker compose build sqlnotify
+          docker build -t sqlnotify:ci .
 
-      - name: Run tests
-        run: make test
+      - name: Run tests and collect coverage inside Docker
+        run: |
+          # Create a directory to store the coverage reports
+          mkdir -p coverage_output
+
+          # Run the container and generate both XML and lcov coverage reports
+          docker run --rm \
+            -v "${{ github.workspace }}/coverage_output":/app/coverage_output \
+            sqlnotify:ci \
+            bash -lc "pytest --cov=sqlnotify --cov-report=xml:/app/coverage_output/coverage.xml --cov-report=lcov:/app/coverage_output/lcov.info"
 
       - name: Upload coverage to Codecov
-        run: make coverage
+        uses: codecov/codecov-action@v4
+        with:
+          files: ./coverage_output/coverage.xml
+          fail_ci_if_error: true
         env:
           CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
 

CONTRIBUTING.md

@@ -49,19 +49,6 @@ make test
 
 # Or directly with pytest
 uv run pytest
-
-# With coverage report
-make test_coverage
-```
-
-### Run specific test files
-
-```bash
-# SQLite tests
-make test_sqlite
-
-# PostgreSQL tests
-make test_postgres
 ```
 
 ## Code Style

Makefile

@@ -16,14 +16,6 @@ publish:
 	@echo "Published successfully"
 
 
-GIT_SHA := $(shell git rev-parse HEAD 2>/dev/null || :)
-
-coverage:
-	@echo "Running coverage (inside Docker)..."
-	@docker compose run --remove-orphans sqlnotify bash -c "$(UV) run pytest --cov=sqlnotify --cov-report=xml && if [ -n '$(GIT_SHA)' ]; then $(CODECOV_CLI) upload-process -t $(CODECOV_TOKEN) -f coverage.xml -C $(GIT_SHA); else echo 'Skipping codecov upload: missing git commit sha'; fi"
-	@echo "Coverage report generated at coverage.xml"
-
-
 release: build
 	@echo "Running release script..."
 	@bash scripts/release.sh $(ARGS)