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

Compile the index selector (non-negative) #61

Closed
V0ldek opened this issue Nov 26, 2022 · 2 comments
Closed

Compile the index selector (non-negative) #61

V0ldek opened this issue Nov 26, 2022 · 2 comments
Assignees
@zwerdlds
Labels
area: selector Support for a JSONPath selector help wanted External contributions welcome type: feature New feature or request
Milestone
v1.0.0

Comments

@V0ldek
Copy link
Member

V0ldek commented Nov 26, 2022

Is your feature request related to a problem? Please describe.
After support is added to the parser in #60, the compiler nmeeds to recognise the new node and compile it into the automaton.

Describe the solution you'd like
The automaton can't do much with the selector, it should just compile it as usual to either a recursive or direct state. The type of transition will be different – we need to introduce an "index" variant of transition, alongside existing Label.
@V0ldek V0ldek added type: feature New feature or request help wanted External contributions welcome area: selector Support for a JSONPath selector mod: compiler labels Nov 26, 2022
@github-actions github-actions bot added the acceptance: triage Waiting for owner's input label Nov 26, 2022
@github-actions
Copy link

Tagging @V0ldek for notifications

@V0ldek V0ldek mentioned this issue Dec 3, 2022
Closed
@V0ldek V0ldek added this to the v1.0.0 milestone Dec 3, 2022
@github-actions github-actions bot added acceptance: go ahead Reviewed, implementation can start and removed acceptance: triage Waiting for owner's input labels Dec 3, 2022
@V0ldek V0ldek mentioned this issue Dec 3, 2022
Closed
4 tasks
@zwerdlds zwerdlds self-assigned this Apr 20, 2023
@V0ldek V0ldek moved this from Todo to Committed in Active rsonpath development Apr 20, 2023
@zwerdlds zwerdlds moved this from Committed to In Progress in Active rsonpath development Apr 20, 2023
@V0ldek
Copy link
Member Author

V0ldek commented May 22, 2023

Completed in #132

@V0ldek V0ldek closed this as completed May 22, 2023
@github-project-automation github-project-automation bot moved this from In Progress to Merged in Active rsonpath development May 22, 2023
@github-actions github-actions bot removed the acceptance: go ahead Reviewed, implementation can start label May 22, 2023
@V0ldek V0ldek moved this from Merged to Released in Active rsonpath development Aug 2, 2023
@V0ldek V0ldek moved this from Todo to Released in Active rq development Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zwerdlds zwerdlds

Labels
area: selector Support for a JSONPath selector help wanted External contributions welcome type: feature New feature or request
Projects
Active rq development
Status: Released
Milestone
v1.0.0
Development

When branches are created from issues, their pull requests are automatically linked.

60-index-selector-non-negative zwerdlds/rsonpath
2 participants
@zwerdlds @V0ldek