Call the stateManager a router, like in Ember core #658

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
4 participants
Contributor

joliss commented Jan 24, 2013

Purely cosmetic, for readability.

Member

darthdeus commented Feb 23, 2013

Can we merge this?

Member

lukemelia commented Feb 23, 2013

I am in favor of the variable name change, but curious: Is this injection style still supported in Ember with the advent of container?

Contributor

joliss commented Feb 24, 2013

Yes, that changed. I think the code that is changed in this PR is code paths that run against deprecated parts of Ember core, so it'll go away eventually. Still, no reason not to make it consistent. :)

Owner

igorT commented Apr 8, 2013

Thanks, seems good.

@igorT igorT closed this Apr 8, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment