Skip to content
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

Version 0.7.2 breaks Glimmer.js app #146

Open
tschoartschi opened this issue Feb 18, 2019 · 4 comments
Open

Version 0.7.2 breaks Glimmer.js app #146

tschoartschi opened this issue Feb 18, 2019 · 4 comments

Comments

@tschoartschi
Copy link

We used ember-service-worker with our Glimmer.js app and it just worked fine with version v0.6.2 now we tried to upgrade to the newest version which is currently v0.7.2 but this upgrade broke our build. This is due to the fact, that Glimmer.js has not installed ember-cli-babel and therefore the following code breaks:

_babelTranspile(tree) {

On the other hand adding ember-cli-babel breaks our Glimmer.js build. So I'm not sure how to resolve this issue.

I know that Ember.js is not Glimmer.js and therefore I'm not sure if this is an issue with ember-service-worker or a Glimmer.js issue. Nevertheless it would be great to share as much addons and plugins as possible between Ember.js and Glimmer.js

Just let me know what you think about this issue

@tschoartschi
Copy link
Author

@marten-dockyard could someone give me feedback on this issue? If we want to support glimmer.js and if we have a rough idea how to do so, I could try to create a PR 🙂

@tschoartschi
Copy link
Author

would be great to get some opinion about this problem. As I said, I could maybe help to make this addon available for glimmer.js

@eshtadc
Copy link
Contributor

eshtadc commented Mar 8, 2019

Looks like this would have started here: 9d6db8a#diff-61e7a77a0dda836b877cba3567f38581
My understanding is that this choice was made to leverage ember-cli-babel various features to work with targets, polyfills, etc. I must admit that I haven't had a chance to work much with glimmer. I don't see anything immediately in glimmer that prevents the usage of ember-cli-babel. Can you explain your scenario a bit more? Do you have any suggestions for alternate approaches?

@tschoartschi
Copy link
Author

@eshtadc thanks for the response :-) if I add ember-cli-babel to our glimmer.js project we can not build it anymore. Maybe it's a bug with the glimmer-application-pipeline. I need to investigate this in more detail maybe I need to open an issue at the glimmer-application-pipeline as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants