Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make Volume classes immutable and create copies of *Container classes in fixtures #234

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

dcermak
Copy link
Owner

@dcermak dcermak commented Oct 29, 2024

No description provided.

@dcermak dcermak force-pushed the volumes-classes-immutable branch from 1354de1 to fb41b59 Compare October 29, 2024 15:22
Copy link

codecov bot commented Oct 29, 2024

Codecov Report

Attention: Patch coverage is 98.70968% with 2 lines in your changes missing coverage. Please review.

Project coverage is 95.14%. Comparing base (b2d4c22) to head (1d45389).

Files with missing lines Patch % Lines
pytest_container/volume.py 98.47% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #234      +/-   ##
==========================================
+ Coverage   94.89%   95.14%   +0.24%     
==========================================
  Files           9       10       +1     
  Lines        1117     1174      +57     
  Branches      164      169       +5     
==========================================
+ Hits         1060     1117      +57     
- Misses         42       43       +1     
+ Partials       15       14       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@dcermak dcermak force-pushed the volumes-classes-immutable branch from fb41b59 to 23f2873 Compare October 29, 2024 22:18
@dcermak dcermak changed the title Volumes classes immutable Make Volume classes immutable and create copies of *Container classes in fixtures Oct 30, 2024
@dcermak dcermak force-pushed the volumes-classes-immutable branch from 23f2873 to 1d45389 Compare October 30, 2024 09:13
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.

1 participant