Skip to content

Ignore coverage artifacts - #471

Merged
negz merged 1 commit into
modelplaneai:mainfrom
dennis-upbound:dennis/gitignore-coverage
Sep 28, 2026
Merged

negz merged 1 commit into
modelplaneai:mainfrom
dennis-upbound:dennis/gitignore-coverage

Conversation

@dennis-upbound

Copy link
Copy Markdown
Collaborator

Description of your changes

Running coverage over a function's tests drops a .coverage database in the function directory. Nothing ignores it, so git add on that directory sweeps the database into the commit as a binary blob, and it takes a reviewer opening the diff to notice. Caught two of them on #470.

Also ignores .coverage.* (what parallel mode writes) and htmlcov/.

I have:

🤖 Generated with Claude Code

Running coverage over a function's tests drops a .coverage database in the
function directory. Nothing ignores it, so `git add` on that directory sweeps
the database into the commit as a binary blob, and it takes a reviewer opening
the diff to notice.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Signed-off-by: Dennis Ramdass <dennis@upbound.io>
@negz
negz merged commit 68f5c89 into modelplaneai:main Sep 28, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants