achambers / ember-cli-deploy-original

An Ember-CLI addon for lightening fast deployment of applications
77 stars 18 forks source link

ember activate breaking changes? #59

Closed EmergentBehavior closed 10 years ago

EmergentBehavior commented 10 years ago

After upgrading to v0.0.5, I noticed two breaking changes:

If these are now expected behaviors, there should be some note indicating such as I was confused why my existing code wasn't working.

achambers commented 10 years ago

Oh crap. Sorry mate. I have been trying to keep the releases up to date. But I accidently marked the PR for that change as [ENHANCEMENT] instead of [BREAKING CHANGE]. Have fixed it up now.

https://github.com/achambers/ember-cli-deploy/releases/tag/v0.0.5

achambers commented 10 years ago

Also, I am yet to update https://github.com/achambers/fuzzy-wookie to work with these changes

EmergentBehavior commented 10 years ago

Good to know :). The changes do make my example in #28 invalid, though. Don't have time to make a docker, etc. now but I can post a new example somewhere or in a wiki if you think it'd be helpful.

achambers commented 10 years ago

My thoughts are that while I've provided an express app to serve the index.html, anyone could provide any other means. If you did that, we could reference it in the README. In planning on writing a ruby version as well.

achambers commented 10 years ago

Can we close this issue now?

EmergentBehavior commented 10 years ago

Sure

On Nov 29, 2014, at 2:47 AM, Aaron Chambers notifications@github.com wrote:

Can we close this issue now?

— Reply to this email directly or view it on GitHub.