You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@izmalk made a great suggestion in #79 around making lower case vocabulary entries be captured for upper case usage (for use at the start of sentences). This made me think that plurals are also not accounted for - and terms that can be pluralised should also be captured.
Edit all lower case vocabulary terms to accept upper case using selective syntax like, e.g. [Uu]ppercase
Full case insensitivity can be a bad idea, as it will allow CAPSLOCK/PascalCase/camelCase usage.
Allow all terms that can be pluralised accept the plural versions as well, e.g. [Nn]namespace(?:s)?
It would be ideal to use non-capturing groups for plurals, as it would be best to use common syntax, and they will speed up processing compared to capturing groups.
It could also be a good idea to accept other parts of speech as well.
The text was updated successfully, but these errors were encountered:
@izmalk made a great suggestion in #79 around making lower case vocabulary entries be captured for upper case usage (for use at the start of sentences). This made me think that plurals are also not accounted for - and terms that can be pluralised should also be captured.
Edit all lower case vocabulary terms to accept upper case using selective syntax like, e.g.
[Uu]ppercase
Allow all terms that can be pluralised accept the plural versions as well, e.g.
[Nn]namespace(?:s)?
The text was updated successfully, but these errors were encountered: