-
Notifications
You must be signed in to change notification settings - Fork 13
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
Overriding with context manager (one and multiple providers) #53
Overriding with context manager (one and multiple providers) #53
Conversation
@nightblure looks great, thanks! |
thanks for the review! corrected according to your comments, please take a look |
hmm, there is still a question about updating the documentation. I can try to add the relevant sections myself if you allow, or you will do this |
@nightblure would be great if you'll add docs as well, but it's up to you. Anyway, thank you for your contribution! |
ok I'll try to do it. within the current MR or separate? |
@nightblure better in separate, I'll merge this |
I implement two little features for ease of use during tests: