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

Correctly import deprecate from @ember/debug #1066

Merged
merged 1 commit into from
Jul 20, 2021

Conversation

mydea
Copy link
Contributor

@mydea mydea commented Jul 19, 2021

This was incorrectly imported from @ember/application/deprecations, which seemed to work, but does not work in current canary - I got this error in an addon testing on canary:

Global error: Uncaught Error: Could not find module `@ember/application/deprecations` 
imported from `@ember/test-helpers/setup-rendering-context`

Anyway, even if this might/should be fixed in canary, the import is wrong and can be updated to the correct place.

@mydea
Copy link
Contributor Author

mydea commented Jul 19, 2021

Canary still fails because of:
#1067

@simonihmig
Copy link
Contributor

Nice, thanks for the quick fix here @mydea 👍

I am also seeing basically any addon test suite failing on canary. Strangely the GH Actions job is not failing immediately, but timing out after running >5h! 🙈.
Same here in this repo

So for both addon maintainers and Github container utilization getting this landed would be very helpful! 😆

@NullVoxPopuli
Copy link
Sponsor Collaborator

I just experienced the same thing, but I have my timeout set much lower :)

@sandydoo
Copy link

Thanks, @mydea. Linking #1068.

@rwjblue rwjblue linked an issue Jul 20, 2021 that may be closed by this pull request
@rwjblue rwjblue merged commit 9cf28a2 into emberjs:master Jul 20, 2021
@rwjblue
Copy link
Member

rwjblue commented Jul 20, 2021

Thanks!

@mydea
Copy link
Contributor Author

mydea commented Jul 26, 2021

Any chance you could cut a release with this change? Would be great to get canary tests passing again for addons :)

buschtoens added a commit to buschtoens/ember-service-helper that referenced this pull request Jul 27, 2021
buschtoens added a commit to buschtoens/ember-service-helper that referenced this pull request Jul 27, 2021
jherdman added a commit to evoactivity/ember-svg-jar that referenced this pull request Oct 1, 2021
jherdman added a commit to evoactivity/ember-svg-jar that referenced this pull request Oct 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing deprecate import on canary
5 participants