Skip to content

Commit

Permalink
ref: disable branch coverage (#81140)
Browse files Browse the repository at this point in the history
this disables branch coverage (coverage of jumps for loops / if
statements / lambdas / try-except / etc.) but drastically improves CI
time

we can revert this out once
nedbat/coveragepy#1746 is resolved and we're
on a sufficiently new-enough version of coverage

<!-- Describe your PR here. -->
  • Loading branch information
asottile-sentry authored Nov 21, 2024
1 parent 4395633 commit 445442f
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 1 deletion.
1 change: 1 addition & 0 deletions requirements-dev-frozen.txt
Original file line number Diff line number Diff line change
Expand Up @@ -182,6 +182,7 @@ s3transfer==0.10.0
selenium==4.16.0
sentry-arroyo==2.16.5
sentry-cli==2.16.0
sentry-covdefaults-disable-branch-coverage==1.0.2
sentry-devenv==1.13.0
sentry-forked-django-stubs==5.1.1.post1
sentry-forked-djangorestframework-stubs==3.15.1.post2
Expand Down
1 change: 1 addition & 0 deletions requirements-dev.txt
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ sentry-devenv>=1.13.0
devservices>=1.0.3

covdefaults>=2.3.0
sentry-covdefaults-disable-branch-coverage>=1.0.2
docker>=7
time-machine>=2.16.0
honcho>=2
Expand Down
4 changes: 3 additions & 1 deletion setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,9 @@ extension =
[coverage:run]
omit =
src/sentry/migrations/*
plugins = covdefaults
plugins =
covdefaults
sentry_covdefaults_disable_branch_coverage

[coverage:report]
# Setting this to 0 makes it falsy, and it gets ignored, so we set it to
Expand Down

0 comments on commit 445442f

Please sign in to comment.