Skip to content
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

Use @types/ember__debug #4173

Merged
merged 1 commit into from
Nov 23, 2021
Merged

Conversation

jherdman
Copy link
Contributor

No need to roll a version of this module when a better one is available. This has the added benefit of allowing assert to assert the test param.

No need to roll a version of this module when a better one is available.
This has the added benefit of allowing `assert` to assert the `test`
param.
@jherdman
Copy link
Contributor Author

I believe these fails aren't due to the changes in this pull request. I think if you modify your ember-try config to use ember-auto-import v2 you'll fix these scenarios on your master branch too.

@k-fish
Copy link
Member

k-fish commented Nov 23, 2021

@jherdman Yeah I was just looking, it's because Ember 4.0 came out and requires auto-import 2. It's a breaking change though since auto import 2.0 isn't backwards compatible as far as I am aware.

Copy link
Member

@k-fish k-fish left a comment

Choose a reason for hiding this comment

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

It passes the rest of the scenarios, so at least the next minor version will be fixed 👍 . We will probably address Ember 4 (and the auto import changes) in another PR.

@AbhiPrasad AbhiPrasad merged commit 8218578 into getsentry:master Nov 23, 2021
@jherdman jherdman deleted the use-types-ember-debug branch November 23, 2021 19:08
onurtemizkan pushed a commit that referenced this pull request Dec 19, 2021
No need to roll a version of this module when a better one is available.
This has the added benefit of allowing `assert` to assert the `test`
param.

Will address Ember 4 (and the auto import changes) in another PR.
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