Code review comment for lp:~pedro.baeza/account-payment/6.1-set_done_optimization

Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

Thanks for the review, Joël. Indeed, when I made this MP, we haven't discussed yet about keyword argumentsap. We are refining us each day ;)

I have made also some PEP8 refactorisation, but there is one context variable that I can't put as keyword because super method (in account_payment module) has *args in the signature.

Please make the merge when you approve the MP, because it has been a lot of time since I made the proposal.

Regards.

« Back to merge proposal