Allow es6 plugins from CLI

peggyjs / peggy

Peggy: Parser generator for JavaScript

https://peggyjs.org/

MIT License

926 stars 65 forks source link

Closed hildjj closed 3 months ago

hildjj commented 3 months ago

Currently, the CLI does:

mod = require(id);

for each plugin.

Come up with a way to designate a plugin as es6.
Change the .action((inputFiles, opts) => { to async
Change parse to parseAsync, and follow up the change in peggy.js and the tests
If plugin is es6, await import(id)

hildjj commented 3 months ago

Didn't need a way to designate a plugin as ES6. await import(plugin) seems to always work.

hildjj commented 3 months ago

Sorry, I think I was supposed to tag @pjmolina. (on #535. going too fast today, apologies)