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

Construct operator tags with state #172

Closed
wants to merge 1 commit into from

Conversation

rkaminsk
Copy link
Contributor

@rkaminsk rkaminsk commented Sep 1, 2023

Optionally, support constructing operator tags from the parse state and an input iterator.

Notes

  • I did not know how to add a state into the verify construction of the exiting operator tests so I simply added tests separately.
  • Maybe you have a better idea how to test this? It would be quite hard for me to dive into the details here.
  • See also location tracking for operators #170.

@foonathan
Copy link
Owner

I've merged it with changes in 53b61f0. Thanks.

@foonathan foonathan closed this Sep 26, 2023
@rkaminsk
Copy link
Contributor Author

Thanks, to you. 👍

@rkaminsk rkaminsk deleted the feature/op-state branch September 26, 2023 18:17
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.

2 participants