-
Notifications
You must be signed in to change notification settings - Fork 705
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
Kusto #2552
Kusto #2552
Conversation
Co-authored-by: Jean Abou-Samra <jean@abou-samra.fr>
Co-authored-by: Jean Abou-Samra <jean@abou-samra.fr>
I've made the suggested changes. Let me know if you want me to squash any commits. |
@Iain-S I've pushed a commit, mostly to make the code style more like the rest of the code base (note that we do not use Black). I've also noticed that this state: "multi_string": [
(r"[^`]+", String),
(r"```", String, "#pop"),
], does not allow single backticks inside |
True, that was an oversight. I've added your suggested tuple and a test to confirm that it works. |
LGTM now, thank you. |
Summary
Add lexer for Kusto.
Also corrects a minor typo in the docs.
Checks
tox
tests passtox -e mapfiles
Linting
Closes
This (hopefully) closes #2098
Further Work \ Known Limitations
1e5
, which we don't currently support.datetime(2015-12-31)
should all be treated as a single literal but we simply markdatetime
as a keyword.h
orH
but we ignore this and would treath/H
as a name.