-
Notifications
You must be signed in to change notification settings - Fork 4
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(BaseClient): prevent raising an error if None is passed to the log_dir #148
Conversation
…d to extract info the duplicate if test for `_metadata is None` is now limited to a single location
patching them will overwrite attributes
this will prevent cassettes to be invalidated due to the metadata end_timestamp changing Addresses #139
a63f737
to
ad6eb3f
Compare
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.
The code is fine and fulfills the requirements of the proposed change. On the other hand, I wonder why you decided not to assert the correctness of the declared log directory by requiring it to be an optional Path instead of the optional string.
@maciej-adamiak the current approach is that we will forcefully create the path if it does not exist: Lines 59 to 60 in 289f4c5
|
No description provided.