You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There is no documentation of what patch_all actually does, even though it's included in an example in README.md.
This is particularly important due to #100, talking about the potential of surprises from patch_all
Does patch_all patch from a whitelist of known libraries that can be patched. Does it have some magic generic approach to try and patch all modules?
Looking at the code makes it clear patch_all patches from a whitelist, but it would be good for this to be documented.
Also, users of patch have to read code to understand what they can patch with it. Even a short piece of documentation linking into the code would lower the barrier of entry.
The text was updated successfully, but these errors were encountered:
There is no documentation of what
patch_all
actually does, even though it's included in an example in README.md.This is particularly important due to #100, talking about the potential of surprises from
patch_all
Does
patch_all
patch from a whitelist of known libraries that can be patched. Does it have some magic generic approach to try and patch all modules?Looking at the code makes it clear patch_all patches from a whitelist, but it would be good for this to be documented.
Also, users of
patch
have to read code to understand what they can patch with it. Even a short piece of documentation linking into the code would lower the barrier of entry.The text was updated successfully, but these errors were encountered: