- Issue created by @danflanagan8
- Status changed to Needs review
over 1 year ago 2:18pm 9 May 2023 - last update
over 1 year ago 71 pass, 1 fail - ๐บ๐ธUnited States danflanagan8 St. Louis, US
Here's a patch with a new test case. I also noticed a bug in two existing test cases. I think they will fail now as well. I can fix those at the same time.
- ๐ฎ๐ณIndia keshavv India
keshav.k โ made their first commit to this issueโs fork.
- Status changed to RTBC
over 1 year ago 7:01am 10 May 2023 - ๐ฎ๐ณIndia keshavv India
Tested the patch, Confirmed that it is working perfect.
Created the MR for easy merge. Moving to RTBC. - ๐บ๐ธUnited States danflanagan8 St. Louis, US
@keshav.k, that's a fail test.
There's no fix.
You are lying. You clearly didn't test this.
Please stay out of the issue queue unless you are going to do real work.
- Status changed to Needs work
over 1 year ago 12:46pm 10 May 2023 -
danflanagan8 โ
committed cc444587 on 2.0.x
Issue #3359154 by danflanagan8: Clue references should be case...
-
danflanagan8 โ
committed cc444587 on 2.0.x
- Status changed to Fixed
4 months ago 7:55pm 27 August 2024 - ๐บ๐ธUnited States danflanagan8 St. Louis, US
I just pushed a fix that simplifies things by converting all text to uppercase before looking for the reference patterns. Because of the post-processing that was even easier than trying to use the
i
flag on the regex patterns. Automatically closed - issue fixed for 2 weeks with no activity.