Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add case sensitivity option or document case insensitivity #13

Open
axelpale opened this issue Jan 18, 2024 · 0 comments
Open

Add case sensitivity option or document case insensitivity #13

axelpale opened this issue Jan 18, 2024 · 0 comments

Comments

@axelpale
Copy link

Love the library. However, it was a painful experience to find out that the trie match is case insensitive. It seems that our system has skipped many workloads over the years that happened to have a lowercase name. The cause was that the workload with the uppercase name had already ran previously.

Therefore, please, add a case sensitivity option. And if that is not possible, maybe document it visibly so that users know what to expect.

The culprit line seems to be: src/index.js#L46

There is also a pending PR #11 from another user that implements the feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant