Looks good, thanks!
A couple of minor corrections inline.
I also noticed in the diff against your previous version of the branch that some line lengths were busting the 79 limit. Looking now, there are some cases I missed before, sorry. Please could you adjust to keep to the line length limit?
« Back to merge proposal
Looks good, thanks!
A couple of minor corrections inline.
I also noticed in the diff against your previous version of the branch that some line lengths were busting the 79 limit. Looking now, there are some cases I missed before, sorry. Please could you adjust to keep to the line length limit?