Code review comment for lp:~edwin-grubbs/launchpad/bug-663861-product-series-timeout-part2

Revision history for this message
Abel Deuring (adeuring) wrote :

Hi Ediwn,

a nice branch! I just wondered if it makes sense to add comment to substitue_filled_numbers() explaining that '~' > c is always true, where c is any character that can appear in a series name.

review: Approve (code)

« Back to merge proposal