Message7442
Hi Thomas:
I have started on this. My first commit:
changeset: 6593:e70e2789bc2c
removed all the magic numbers 25, 30 from the code and replaced them with references to
indexer.maxlength. I added testing for the code paths and sent it to CI.
If nothing else, this reduces your diff to the one liner:
- self.maxlength = 25
+ self.maxlength = 50
I'll try to get to the rest of the changes (db migration, modify maxlength, tests) tomorrow
or the next day I hope. |
|
Date |
User |
Action |
Args |
2022-01-25 18:27:18 | rouilj | set | messageid: <1643135238.26.0.936446318176.issue2551189@roundup.psfhosted.org> |
2022-01-25 18:27:18 | rouilj | set | recipients:
+ rouilj, ThomasAH |
2022-01-25 18:27:18 | rouilj | link | issue2551189 messages |
2022-01-25 18:27:18 | rouilj | create | |
|