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

pat: Implement prefix pattern #62

Closed
timbray opened this issue Jun 7, 2022 · 0 comments · Fixed by #190
Closed

pat: Implement prefix pattern #62

timbray opened this issue Jun 7, 2022 · 0 comments · Fixed by #190

Comments

@timbray
Copy link
Owner

timbray commented Jun 7, 2022

Note that the following are equivalent

{ "a": [ {"shellstyle": "foo*"} ] }
{"a": [ {"prefix": "foo"} ] }

It's already special-cased in makeShellStyleAutomaton. So there are a variety of ways this could be done, ranging from just re-writing the pattern to fixing up the switch on ValueType in AddPattern. But no new automaton construction should be required.

@timbray timbray changed the title api: Implement prefix pattern pat: Implement prefix pattern Jan 26, 2023
timbray added a commit that referenced this issue Jan 28, 2023
fixes: #62
Signed-off-by: Tim Bray <tbray@textuality.com>
timbray added a commit that referenced this issue Jan 28, 2023
fixes: #62
Signed-off-by: Tim Bray <tbray@textuality.com>
timbray added a commit that referenced this issue Jan 31, 2023
fixes: #62
Signed-off-by: Tim Bray <tbray@textuality.com>
timbray added a commit that referenced this issue Feb 16, 2023
fixes: #62

Signed-off-by: Tim Bray <tbray@textuality.com>
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

Successfully merging a pull request may close this issue.

1 participant