Skip to content

Small fixes after some more experimentation #48

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
Merged
merged 3 commits into from
Feb 17, 2025
Merged

Small fixes after some more experimentation #48

merged 3 commits into from
Feb 17, 2025

Conversation

ruoso
Copy link
Collaborator

@ruoso ruoso commented Feb 17, 2025

The changes here are:

  1. Read transitions also need to pass the data forward, that way we can keep context across messages
  2. Give an error message explaining where the parse failed.
  3. Support parsing source code directly, instead of just via a file name

ruoso added 3 commits February 17, 2025 15:39
@ruoso
Change how data is handled on read transitions
e8675d0 
This allows data to flow through, as slong as the state callback sends it.
This allows for carrying data across transitions even when it is not read
in that transition
@ruoso
tokenize where parsing stops
bda180d 
useful when the interpreter parser fails
@ruoso
support parsing source code text
c8993cf 
previously you could only provide a file name.
@ruoso ruoso merged commit a0b45f0 into PacktPublishing:main Feb 17, 2025
1 check passed
@ruoso ruoso deleted the fixups branch February 17, 2025 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ruoso