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
Deprecate import Ember from 'ember'
#20686
base: main
Are you sure you want to change the base?
Deprecate import Ember from 'ember'
#20686
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm surprised it didn't treat this file as a move...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My trick for reliably getting git to understand a move: Make a commit with just the file rename and no other changes, even if it leaves the codebase in a broken state for that commit.
@@ -90,6 +90,15 @@ function deprecation(options: DeprecationOptions) { | |||
test the behavior without encountering the deprecated feature, just as users would. | |||
*/ | |||
export const DEPRECATIONS = { | |||
DEPRECATE_IMPORT_EMBER(importName: string) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is and needs to be a function, I hope that's ok 😅
import * as templateCompilation from '@ember/template-compilation'; | ||
|
||
// eslint-disable-next-line @typescript-eslint/no-namespace | ||
namespace Ember { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have a hunch that some of these types may need to move back to the index.ts, but we'll see as the PR progresses
17a6f91
to
b5eaa95
Compare
7a95c95
to
70d0ca8
Compare
…ty is accessed, because we re-export the ember barrel from the template compiler
703a25b
to
3e46fe1
Compare
RFC: emberjs/rfcs#1003
Guide: ember-learn/deprecation-app#1381
Advancement: emberjs/rfcs#1015