standardebooks / tools

The Standard Ebooks toolset for producing our ebook files.
Other
1.43k stars 127 forks source link

y-032 #700

Closed vr8hub closed 6 months ago

vr8hub commented 6 months ago

This looks for text immediately following an italic with an epub:type, excluding (s|es|er). However, the exclusion doesn't have a \b following it, so it excludes anything beginning with (s|es|er), not just s, es, and er themselves.

Was your intent to only exclude s, es, and er, or did you mean to exclude anything beginning with those? If the former, I'll make the adjustment since I'm testing the test for it.

acabal commented 6 months ago

Yes I suppose it makes sense to add \b to be strictly correct, though I don't think this particular test has found any false positives across the corpus despite that.