-
Notifications
You must be signed in to change notification settings - Fork 3
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
Fix mount configuration change check for bind mounts #134
Conversation
docker_mount_for() returns Path for bind mounts, which resulted in a type mismatch even when configuration did not change, and users would get spurious warnings.
Codecov Report
@@ Coverage Diff @@
## main #134 +/- ##
=======================================
Coverage 86.19% 86.19%
=======================================
Files 9 9
Lines 826 826
=======================================
Hits 712 712
Misses 114 114
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
The CI fail is just Codecov failing to upload, re-running will hopefully fix it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect, thanks a lot!
@danielhollas I suggest that we wait with a release until after #132 is merged, but please let me know if you would prefer for me to make a release right away. |
Yes, let's wait for #132, thanks!
|
docker_mount_for() returns Path for bind mounts, which resulted in a type mismatch even when configuration did not change, and users would get spurious warnings.
I modified two tests so that they would catch the bug.
CC @csadorf this is the bug I mentioned in #132.