code_review_notes.txt

   1 Please add unit tests for the new count-only functionality.
   2
   3 line 43 def matchmake:
   4      This was making redundant calls to regex matching functions and so could be slower than necessary. I suggest changes that use the walrus operator to keep the same logical structure without the redundant calls.
   5
   6
   7 line 212 def __init__:
   8
   9      Minor note: This constructor is taking a lot of arguments.  This is fine, but from a style + maintainability perspective it might make sense to create a new class for the regex matching configuration and pass a configuration object to this contructor instead.