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.

Sign up for GitHub

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

Jump to bottom

Bugs in JMESPath lexer #2179

Closed
mondeja opened this issue Jul 8, 2022 · 0 comments · Fixed by #2182
Closed

Bugs in JMESPath lexer #2179

mondeja opened this issue Jul 8, 2022 · 0 comments · Fixed by #2182
Labels
A-lexing area: changes to individual lexers
Milestone
2.13.0

Comments

@mondeja
Copy link
Contributor

mondeja commented Jul 8, 2022 •
edited
Loading

Reported at #2175 (comment)

  • Expression type before subexpression: &(foo)
  • Expression type for identifiers: &foo and &"foo"

Additionally:

  • - characters can be part of identifiers, even unquoted
@mondeja mondeja mentioned this issue Jul 8, 2022
Merged
@mondeja mondeja changed the title Expression type character not handled by JMESPath lexer Bugs in JMESPath lexer Jul 10, 2022
@mondeja mondeja mentioned this issue Jul 10, 2022
Merged
@Anteru Anteru added this to the 2.13.0 milestone Aug 14, 2022
@Anteru Anteru added the A-lexing area: changes to individual lexers label Aug 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-lexing area: changes to individual lexers
Projects
None yet
Milestone
2.13.0
Development

Successfully merging a pull request may close this issue.

Fix some cases in JMESPath lexer mondeja/pygments
2 participants
@Anteru @mondeja