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

Fix inline group parsing #207

Merged
merged 1 commit into from
Aug 22, 2023
Merged

Fix inline group parsing #207

merged 1 commit into from
Aug 22, 2023

Conversation

jrandolf-2
Copy link
Contributor

@jrandolf-2 jrandolf-2 commented Aug 21, 2023

When a new subparser is constructed, we must pass in known groupnames and typenames so that the subparser also knows about them. It's also imperative that the main parser appends all unknown rule identifiers found by the subparser so we can reparse them later.

When a new subparser is constructed, we must pass in known `groupnames` and `typenames` so that the subparser also knows about them. It's also imperative that the main parser appends all unknown rule identifiers found by the subparser, so we can reparse them later.
@jrandolf-2
Copy link
Contributor Author

jrandolf-2 commented Aug 21, 2023

@anweiss After we merge this, could we make a release? We have a package that we want to publish, but it depends on main (not a version number), so we cannot publish it until cddl does one release.

@anweiss anweiss merged commit 606bd0e into anweiss:main Aug 22, 2023
@jrandolf-2 jrandolf-2 deleted the jrandolf/sub-parse branch August 22, 2023 16:40
@jrandolf-2
Copy link
Contributor Author

@anweiss Do you plan to make a release? Again, we can't publish to crates.io because crates.io doesn't allow packages depending on a git branch (we are currently depending on main).

@anweiss
Copy link
Owner

anweiss commented Aug 28, 2023

Thanks for your patience @jrandolf. 0.9.3 has been published to crates.io. Full release is here.

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