Regexp identifies months as a part of the word about datefinder

[itdxer]

I've tried to modify example and when I modify this line (for DATES_PATTERN variable)

(?P<months>{months})

to this

(?P<months>{months})(?P<month_ending>\W|$)

it solves the problem.

In case if this solution is fine - I can make a pull request.

P.S. All tests work fine. Also I can add a few more tests to make sure that this problem will not occur later.

[itdxer]

There also exists the same issue related to timezones

E           assert 't at 2p.m. m...ndard time on' == 'at 2p.m. moun...ndard time on'
E             - t at 2p.m. mountain standard time on
E             ? --
E             + at 2p.m. mountain standard time on
tests/test_extract_date_strings.py:20: AssertionError

for

['the big fight at 2p.m. mountain standard time on ufc.com', 'at 2p.m. mountain standard time on']

it cuts letter t, because it's one of the choices

[akoumjian]

Go ahead and put up a PR. Just want to make sure we aren't removing other valid matches before merging it in. Great work!

[itdxer]

I've solve issue related to month, but I failed to solve it for extra_tokens.

It still extracts

t at 2p.m. mountain standard time on ufc.com

from

the big fight at 2p.m. mountain standard time on ufc.com

The same trick didn't help for this case.

[mnba]

In fact such ending-guards ( "...\W|$" ) will be needed for DateFinder.REPLACEMENTS strings too, since source replacement can be easily done within words, corrupting them. Doing tokenization would be better approach, though very different one

[akoumjian]

This was resolved.