Conversation
Greptile SummaryThis PR adds Codecov integration by wiring up
Confidence Score: 4/5Safe to merge after fixing the TOML section headers in One P1 issue: the
Important Files Changed
Sequence DiagramsequenceDiagram
participant GH as GitHub Actions
participant COV as coverage.py
participant CC as Codecov
GH->>COV: coverage run -m pytest (all events: push + PR)
COV-->>GH: coverage.xml
GH->>CC: codecov/codecov-action@v6 (upload coverage.xml)
CC-->>GH: Coverage report posted to PR
Reviews (2): Last reviewed commit: "Update ci.yml" | Re-trigger Greptile |
| [run] | ||
| branch = true | ||
| source = [ | ||
| 'dimos', | ||
| ] | ||
|
|
||
| [report] | ||
| exclude_also = [ | ||
| 'if TYPE_CHECKING', | ||
| 'assert False', | ||
| ': \.\.\.(\s*#.*)?$', | ||
| '^ +\.\.\.$', | ||
| 'pytest.fail\(' | ||
| ] |
There was a problem hiding this comment.
Wrong TOML section headers — config silently ignored
Coverage.py auto-discovers .coveragerc.toml as a TOML file, but when read as TOML it expects [tool.coverage.run] / [tool.coverage.report] section headers. The bare [run] and [report] headers shown here are INI-format conventions; they are not recognized under TOML parsing, so branch = true, source, and all exclude_also patterns are silently dropped.
| [run] | |
| branch = true | |
| source = [ | |
| 'dimos', | |
| ] | |
| [report] | |
| exclude_also = [ | |
| 'if TYPE_CHECKING', | |
| 'assert False', | |
| ': \.\.\.(\s*#.*)?$', | |
| '^ +\.\.\.$', | |
| 'pytest.fail\(' | |
| ] | |
| [tool.coverage.run] | |
| branch = true | |
| source = [ | |
| 'dimos', | |
| ] | |
| [tool.coverage.report] | |
| exclude_also = [ | |
| 'if TYPE_CHECKING', | |
| 'assert False', | |
| ': \.\.(\s*#.*)?$', | |
| '^ +\.\.\.$', | |
| 'pytest.fail\(' | |
| ] |
Add codecov, so we can easily see coverage changes in each PR.