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

Update use of modularized grammars #298

Open
thec0keman opened this issue Aug 23, 2017 · 0 comments
Open

Update use of modularized grammars #298

thec0keman opened this issue Aug 23, 2017 · 0 comments

Comments

@thec0keman
Copy link
Collaborator

thec0keman commented Aug 23, 2017

PegJS is working on support for modularized grammars. Once this is finalized and released, it would be good to update emblem to take advantage of this (over against use of pegjs-import).

There are a couple of issues with our current structure:

  1. modules are copied when imported, so emblem's size grows as you reuse a module
  2. circular dependencies do not work, which has resulted in some less than intuitive module structure. (In general, it forces pushing shared code higher in the chain).

Tracking issue.

@thec0keman thec0keman added this to the v1.0.0 milestone Feb 14, 2018
@thec0keman thec0keman removed this from the v1.0.0 milestone Dec 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant