Skip to content

Deprecate import Ember from 'ember' - #20686

Merged
ef4 merged 2 commits into
emberjs:mainfrom
NullVoxPopuli:deprecate-import-ember-from-ember
May 10, 2024
Merged

ef4 merged 2 commits into
emberjs:mainfrom
NullVoxPopuli:deprecate-import-ember-from-ember

Conversation

@NullVoxPopuli

@NullVoxPopuli NullVoxPopuli commented Apr 22, 2024 •

Copy link
Copy Markdown
Contributor

Comment thread packages/ember/barrel.ts Resolved
Comment thread packages/ember/barrel.ts Resolved
@NullVoxPopuli
NullVoxPopuli force-pushed the deprecate-import-ember-from-ember branch 2 times, most recently from 17a6f91 to b5eaa95 Compare April 29, 2024 21:57
@NullVoxPopuli
NullVoxPopuli force-pushed the deprecate-import-ember-from-ember branch from 7a95c95 to 70d0ca8 Compare May 2, 2024 15:10
@NullVoxPopuli
NullVoxPopuli marked this pull request as ready for review May 2, 2024 15:17
@NullVoxPopuli
NullVoxPopuli force-pushed the deprecate-import-ember-from-ember branch from 703a25b to 3e46fe1 Compare May 9, 2024 21:10
Comment thread packages/ember/tests/reexports_test.js Resolved
@kategengler
kategengler requested a review from ef4 May 9, 2024 23:43
@NullVoxPopuli
NullVoxPopuli force-pushed the deprecate-import-ember-from-ember branch from 9cfcb7b to b00fbb3 Compare May 10, 2024 02:29

@kategengler kategengler left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd like @ef4 to take a look too

@ef4

ef4 commented May 10, 2024

Copy link
Copy Markdown
Contributor

High priority places that currently hit the deprecation in a stock app:

  • ember-cli-app-version
  • ember-resolver
  • @glimmer/component
  • ember-inspector

Chasing all these down and getting them fixed and released is going to be important to make this practical as an until 6.0 deprecation.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants