chore: coverage measurement in CI + README badges, test-count reconcile, all extra - #60
Merged
Conversation
…an `all` extra - ci.yml: run pytest with --cov (no --cov-fail-under, so coverage is measured and printed on every leg but never gates the build); pytest-cov added to dev. - README: add Python-versions and License badges; reconcile the test-count claim (~440, matching CONTRIBUTING) with the default-vs-full-matrix nuance. - CONTRIBUTING: ~430 -> ~440 tests. - pyproject: add an `all` convenience extra (every framework adapter; requires Python >=3.10 and is heavy — documented inline). Local: 514 passed, 2 skipped; total line coverage 63%.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
A low-risk packaging/docs polish pass — no library source changes.
Changes
testjob runspytest --cov=dprovenancekit --cov-report=term-missing:skip-coveredon each matrix leg, andpytest-covis added to thedevextra. There is deliberately no--cov-fail-under, so coverage is visible in the log but never blocks a merge.380+and CONTRIBUTING's~430now both read~440, keeping the honest "a default run skips uninstalled adapters; CI's full matrix runs them all" nuance.allextra: a convenience aggregate that installs every framework adapter at once. Documented inline that it requires Python ≥ 3.10 (openai-agents / llama-index / crewai declare that floor) and is heavy (crewai pulls chromadb/onnxruntime); 3.9 users pick specific extras.Verification (local, Python 3.13)
514 passed, 2 skipped(crewai / openai-agents not installed → correctly skipped).pip install -e ".[dev]"resolves with the newpytest-cov;[all]parses and resolves.Not done (deliberately)
otelextra: OTLP ingestion (otel_ingest.py) is pure standard library, so anopentelemetry-*dependency would have no consumer. The 8 existing framework adapters already each ship an extra.