Dark Mode

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[CLEANUP beta] Remove deprecated Registry and Container behavior#11853

Merged
rwjblue merged 2 commits intoemberjs:masterfrom
dgeb:remove-registry-deprecations
Jul 21, 2015
Merged

[CLEANUP beta] Remove deprecated Registry and Container behavior#11853
rwjblue merged 2 commits intoemberjs:masterfrom
dgeb:remove-registry-deprecations

Conversation

Copy link
Member

dgeb commented Jul 21, 2015

Rebases @cibernox's PR #11756, which performs the following:

  • Registry no longer instantiates a container and forwards lookup / lookupFactory to it.
  • Removed checks for ember-application-instance-initializers and ember-application-initializer-context since they have been enabled by default for ~4 months now.

Also performs the following associated cleanup of deprecated container behavior:

  • Containers no longer create a registry if one is not supplied to the constructor.
  • Containers no longer expose their associated registry's interfaces.

cibernox and others added 2 commits July 21, 2015 15:58
* Registry no longer instantiates a container and forwards
lookup/lookupFactory to it.
* Removed checks for `ember-application-instance-initializers` and `ember-application-initializer-context`
since they have been enabled by default for ~4 months now.
* Containers no longer create a registry if one is not supplied to the
constructor.

* Containers no longer expose their associated registry's interfaces.
dgeb changed the title Remove Registry and Container deprecations [CLEANUP beta] Remove Registry and Container deprecations Jul 21, 2015
dgeb changed the title [CLEANUP beta] Remove Registry and Container deprecations [CLEANUP beta] Remove deprecated Registry and Container behavior Jul 21, 2015
Copy link
Member

rwjblue commented Jul 21, 2015

- Looks good.

We need to decide what to do with this code in ember-test-helpers.

@dgeb - Thoughts?

rwjblue mentioned this pull request Jul 21, 2015
Copy link
Member Author

dgeb commented Jul 21, 2015

@rwjblue so, as we just discussed, that code in ember-test-helpers existed solely to quiet the deprecations (which are now removed with this PR).

We can now test for its need in ember-test-helpers with a method in container check before patching.

Copy link
Member

rwjblue commented Jul 21, 2015

PR submitted upstream: emberjs/ember-test-helpers#85

rwjblue added a commit that referenced this pull request Jul 21, 2015
[CLEANUP beta] Remove deprecated Registry and Container behavior
rwjblue merged commit 5a2ec77 into emberjs:master Jul 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants