-
Notifications
You must be signed in to change notification settings - Fork 28
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
make it an ember-addon :) #29
Comments
I plan to! Good idea :) |
+1 I was literally just trying to figure out how to make this work without adding the file as one of my own components. |
Took a stab at it. https://github.com/runspired/ember-cloaking/tree/es6-conversion I haven't built an addon before and I must be doing something wrong because in my project I still get
|
Got it working, turned out I'd just misunderstood which directory was getting merged with the main app's tree. https://github.com/runspired/ember-cloaking/tree/es6-conversion I've left a non es6 version in the main directory pointed to by bower for the moment, which should be changed to the output of a built version of the addon to remain compatible with non es6/ember-cli users. If this looks good to you, I'll submit a pull request. |
ping @eviltrout Thoughts? This would obviously need updated/redone for the new changes you just made, and a built version would need to be maintained, possibly using https://github.com/thomasboyt/browserify-es6-modules ? |
I'd still like to do this, I just have been spread way too thin lately! |
I needed this last month, so I went ahead and forked and did it. As I mentioned, it would need your new fixes (last 3 commits) and it needs the es6 browserify to be able to continue providing a globals based version for legacy. I'd be more than happy to contribute a PR to continue the discussion / enable review. |
I would love a PR for this if you h ave time! |
Accomplished by #32, this can be closed. |
No description provided.
The text was updated successfully, but these errors were encountered: