Message294031
In general LGTM. I left few minor comments on Rietveld. Needed the versionadded directive, Misc/NEWS and What's New entries.
But shouldn't the new function have name "filterfalse" for parallel with itertools.filterfalse? And this would better match the style of the fnmatch module, fnmatchcase doesn't contain an underscore.
Tests for fnmatch.filter are too poor. I think we should add more tests (this is other issue). |
|
Date |
User |
Action |
Args |
2017-05-20 14:31:54 | serhiy.storchaka | set | recipients:
+ serhiy.storchaka, steven.daprano |
2017-05-20 14:31:54 | serhiy.storchaka | set | messageid: <1495290714.4.0.632365827942.issue30413@psf.upfronthosting.co.za> |
2017-05-20 14:31:54 | serhiy.storchaka | link | issue30413 messages |
2017-05-20 14:31:54 | serhiy.storchaka | create | |
|