ohmjs / ohm

A library and language for building parsers, interpreters, compilers, etc.
MIT License
5.01k stars 217 forks source link

feat: Add proper support for ES modules via a Rollup build step #350

Closed pdubroy closed 2 years ago

pdubroy commented 2 years ago

Fixes #185. Now that Ohm can be built with Rollup (see #344), we can ship a proper ES module via a Rollup build step.

pdubroy commented 2 years ago

@megabuz: Please take a look and let me know if this looks like a reasonable approach to you.

pdubroy commented 2 years ago

Published this as ohm-js@16.2.0-pre.esm.

pdubroy commented 2 years ago

Going to merge this now since it seems to fix the problem. Happy to address anything else in a follow-up.

megabuz commented 2 years ago

Thank you! I'm glad ohm receives better ESM support. I'm going to test it in my project.

It's working, nice!