Skip to content

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