You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Nearley made an internal change with how they handle CLI arguments internally in this commit before release 2.16.0.
Unfortunately, that change breaks this package. The context for handling @include directives does not get correctly passed to Nearley. The change needed to handle this is as follows, in src/index.js+12:
- var compilation = compile(parser.results[0], {file: this.resourcePath});
+ var compilation = compile(parser.results[0], {args: [this.resourcePath]});
But I'm not sure what would be wisest to do about versioning. Perhaps bump the major version here and say that ^2 only works with nearley<2.16.0? It's unfortunate for sure.
The text was updated successfully, but these errors were encountered:
cemulate
changed the title
@include broken for nearley > 2.16.0
@include broken for nearley >= 2.16.0
Dec 10, 2019
Nearley made an internal change with how they handle CLI arguments internally in this commit before release 2.16.0.
Unfortunately, that change breaks this package. The context for handling
@include
directives does not get correctly passed to Nearley. The change needed to handle this is as follows, in src/index.js+12:But I'm not sure what would be wisest to do about versioning. Perhaps bump the major version here and say that
^2
only works withnearley<2.16.0
? It's unfortunate for sure.The text was updated successfully, but these errors were encountered: