Closed ajkerrigan closed 1 year ago
Patch coverage: 100.00
% and project coverage change: +2.86
:tada:
Comparison is base (
b6ebff8
) 87.73% compared to head (5287e23
) 90.59%.:exclamation: Current head 5287e23 differs from pull request most recent head 3092d9c. Consider uploading reports for the commit 3092d9c to get more accurate results
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
Looks like when this plugin checks path deps, it's looking at the
dev
group by default and bringing extras (and their dep trees) into the frozen wheel.Summary of the changes:
main
group (don't freezedev
dependencies).extra
markers in the frozenRequires-Dist:
lines for dependencies that come in via extras. So...pytest-cov
, which depends oncoverage
, which includestomli
as part of thetoml
extra. When we freeze thetomli
dependency, don't include anextra
marker.c7n-gcp
as part of thegcp
extra. Do freezec7n-gcp
with the... and extra == "gcp"
marker.gcp
orazure
markers. Add extra markers for those.This change effectively looks to mirror the utility of requirements.txt files created with
poetry export
, but specifies extras at install time rather than build time. For example:Addresses https://github.com/cloud-custodian/cloud-custodian/issues/8488