Could you add docs and tests?

Do we need to specify the pickler and unpickler classes or just dumps() and loads() functions?

I'd also go fordumps and loads instead of pickler/unpickler classes. With a proper definition (e.g. takes a Python object and returns bytes) I think that should make it much more simpler (it is also similar how other pickle-related APIs work outside of the stdlib)

Doc and test have been added. @merwok

The pickler and unpickler classes have been replaced with dumps() and load() functions. @isidentical @serhiy-storchaka

✅ Deploy Preview for python-cpython-preview canceled.

Should we require the serializer to accept the pickle protocol? It does not make sense for other than pickle formats.

What happens if only serializer or only deserializer are passed?

What happens if only serializer or only deserializer are passed?

elif (serializer is None and deserializer is not None) or (deserializer is None and serializer is not None):
    raise ShelveError("Serializer and deserializer must be defined together.")

In this case, shelve raises an error.

This seems to have been forgotten over end-of-year break in 2022. Do you want to revive it?

The BsdDbShelf class needs to be updated as well; its extra functions should use deserializer rather than Unpickler

I accidentally deleted my forked repository, so I can't update my old PRs. So I opened a new PR.