-
Notifications
You must be signed in to change notification settings - Fork 92
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add an ES5 build #57
Comments
@czosel - I am facing the same issue here, are you planning to create a PR? |
Hi @akshaygolash - I'm not planning on creating a PR for this right now, because I solved the issue differently already. As you can see, this issue is already quite old. I think I'm gonna go ahead and create a issue that asks directly if @TomFrost would accept help with the maintenance of this library - I think Jexl is a great library and it would be a pity if simple things like this issue stop people from using it. Edit: Done, see #82 |
@czosel - I agree with you 100%, Thanks for replying. Do you mind to share the approach you used to fix the issue? The issue for my project is I am using Angular 9, and angular CLI expects all libraries to provide es5 build, and hence it is breaking in IE 11. Any help is appreciated! |
We're using Jexl with Ember.js, and the issue was fixed in the framework itself (ember-auto-import transpiles dependencies automatically, embroider-build/ember-auto-import#41). Sorry I can't be of more help 😉 |
Oh, I see, Thank you for your input. Much appreciated. |
Thanks @TomFrost for your prompt response, Any idea what's the ETA for this enhancement? |
Hi @akshaygolash! No promises, but I'm hoping to cut a release this week. If all goes well, I'll have this in it. |
Awesome, thanks! Much appreciated. |
Any updates? |
Hi @TomFrost . First of all, Thank you for the awesome library. We have recently updated to version 2.2 and our application will not run on older browsers that do not support es2015. Can you please let me know if we can expect a fix for this in the near future. |
Exactly what I need. |
While on the subject of builds, I would love to see a more versatile module format like UMD or just plain es6 modules without the CommonJS stuff in the way. This would be great for people using WebPack and Rollup since we could get tree-shaking working with this library. |
@TomFrost Is this already part of any release now? |
Hi everyone! I've been racing to try to complete the port to Typescript, as that will have to be compiled anyway. This isn't out yet, but is being actively worked on. If my timeline drags on, I'll release with @czosel's PR to tide folks over. |
Hi @TomFrost, any updates on the typescript port? Might be worth sharing even something incomplete, to allow people to contribute? |
Still tinkering on the Typescript side of things-- but a release is long overdue, so many thanks to @czosel's PR to add the ES5 build! That is now released in 2.3.0. |
Awesome, thanks @TomFrost! 🎉 |
Nonono, thank you! 😉 |
Hi @TomFrost
I've been running into some issues with running Jexl 2.0 in older browsers that don't support ES6 yet (specifically IE 11). Many libraries are publishing transpiled (and often also bundled) code to npm. Would you be interested in providing something like that for Jexl as well?
I might be able to open a PR for this, but I figured I'd ask first if you're interested at all 😉
The text was updated successfully, but these errors were encountered: