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

Make the expr syntax explicit in the docs #2351

Closed
wants to merge 1 commit into from

Conversation

igbanam
Copy link

@igbanam igbanam commented Jun 18, 2024

#2350 shows me it's possible for the docs to be misinterpreted. This could lead to errors as explained in the issue. This change makes the syntax more explicit so people can translate for their use cases easier

@tbm
Copy link
Contributor

tbm commented Jun 19, 2024

afaik the quotes are only needed because of a parsing issue; so the real fix would be to solve the parsing issue (#872.)

@igbanam
Copy link
Author

igbanam commented Jun 19, 2024

Thanks for hinting this. I'll take a stab at fixing that but it may take a while.

@igbanam igbanam closed this Jun 19, 2024
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 this pull request may close these issues.

None yet

2 participants