-
Notifications
You must be signed in to change notification settings - Fork 77
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
thoughts on this being an addon? #28
Comments
@stefanpenner I love the idea and it would certainly make my life easier.. Really all I would need is a way to add commands to it and I think it could be done. Instead of One potential problem which I don't know the answer too is adding these commands to it globally. The initial app structure setup I provide would require a global blueprint to use to set up a new project, everything else works in a generated project. With that being said, I may be able to reverse the structure of how I have it now to resolve this. Currently it's cordova folders with an ember one inside, If I could switch it to ember folders with a cordova one inside then you could generate an ember-cli project and run one of my commands after I am currently planning on removing the bower install / npm install parts of this by switching those dependencies to cli addons which would help in this sort of transition |
@stefanpenner I've actually put some work into adding commands within an addon this morning and I'm making good progress. Could probably get a PR ready for ember-cli by the end of the weekend |
@jakecraige im excited :) |
Sounds awesome! |
@jakecraige if you are not already, feel free to pop by #ember-cli on freenode |
@stefanpenner I'm out of the house right now but i should've able to be on On Saturday, June 28, 2014, Stefan Penner notifications@github.com wrote:
Thanks, Jake Craige |
Got the PR into ember-cli that will make this possible, ember-cli/ember-cli#1196 |
💯 |
it would be great to see if we can expose the appropriate hooks in ember-cli proper for this to merely be an addon.
The text was updated successfully, but these errors were encountered: