Update patch set 4

Browse Source

Patch Set 4:

Hello James,

Thanks a lot for your contribution. 

I have reworked the commit logs for your to make it a bit more concise and explain what it does.

Just a note for the future, usually you would want to break down with different patches. For example have the PEP8 fixes in one review and the one that add the feature in a following review (you can chain reviews in gerrit), but that's just fine like this now.

It would be nice if we could have some tests testing the reverse feature properly,

Thanks a lot again,

Patch-set: 4
Label: Verified=0

...

This commit is contained in:

Gerrit User 866 2015-02-03 16:51:24 +00:00 committed by Gerrit Code Review

parent db2c098d9b

commit efc7ec44ff

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Show Stats Download Patch File Download Diff File Expand all files Collapse all files

Diff Content Not Available