Skip to content

Fix traceback because _seen_ops is not initialised#451

Merged
Byron merged 1 commit intogitpython-developers:masterfrom
barry-scott:pr-fix-callable-remote-progress
May 29, 2016
Merged

Fix traceback because _seen_ops is not initialised#451
Byron merged 1 commit intogitpython-developers:masterfrom
barry-scott:pr-fix-callable-remote-progress

Conversation

@barry-scott
Copy link
Contributor

must call the base class init

must call the base class __init__
@Byron Byron added this to the v2.0.4 - Bugfixes milestone May 29, 2016
@Byron
Copy link
Member

Byron commented May 29, 2016

Argh, thanks ! That code is not called by the test-suite yet :( .

@Byron Byron merged commit ff0365e into gitpython-developers:master May 29, 2016
Byron added a commit that referenced this pull request May 29, 2016
Related to #451

Signed-off-by: Sebastian Thiel <byronimo@gmail.com>
def __init__(self, fn):
self._callable = fn

RemoteProgress.__init__(self)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be super(CallableRemoteProgress, self).__init__() instead.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks - it's done.

Byron added a commit that referenced this pull request May 29, 2016
@nvie
Copy link
Contributor

nvie commented May 29, 2016

Cheers!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants