acornjs / acorn-jsx

Alternative, faster React.js JSX parser
MIT License
648 stars 72 forks source link

Acorn 8 compatibility #118

Closed sverweij closed 4 years ago

sverweij commented 4 years ago

hi - acorn-jsx works well with acorn 8 👌 . It doesn't advertise that in its peer dependencies yet, though. This makes yarn & npm complain about unmet peer dependencies.

Also, on the ci acorn-jsx isn't tested against acorn 8 yet.

A few days ago I've taken the liberty to make a PR (#116) that fixes both these. It might have escaped notice because I didn't file an issue for it first - my apologies if that's what's expected; I'm kinda new around here...

Could you have a look and let me know of any concerns?