👍

We could support another, less verbose, way to write. Now:

class MyCallbacks(pygit2.RemoteCallbacks):
    def __init__(self):
        self.credentials = pygit2.UserPass("libgit2", "libgit2")

callbacks = MyCallbacks()

Or less verbose:

callbacks = pygit2.RemoteCallbacks(
    credentials=pygit2.UserPass("libgit2", "libgit2"),
)

That's a good idea. The credentials or certificate callbacks are some which you sometimes must provide in order to complete the operation, so it makes sense to make them easier to provide in-line.

This should now give us parity in both. Through the unification we've gained the progress callbacks on clone and the certificate check on fetch and push, instead of having to implement each of them.