search

jarslab / babel-maven-plugin

Babel transcription plugin for Maven
MIT License
25 stars 8 forks source link

Feature/performance #12

Closed jonaskoperdraat closed 4 years ago

jonaskoperdraat commented 4 years ago

This PR continues on the work of PR #11 (fixes #7), and adds a significant performance increase, whereby it mostly fixes #9. I say mostly, because I think there might still be room for improvement, if we manage to utilize precompiled javascript, instead of having to create a number of ScriptEngines. However, I think this provides for a good start, and a flexible base to build upon if we feel the need to add different transpiling strategies.

Seeing as this builds on #11, it would suffice to merge only this PR, but for the time being, I'll leave both open, so as not to loose any comments.

jonaskoperdraat commented 4 years ago

@milpol, these last commits address all the remaining issues from #11.