Skip to content

update embroider#9597

Merged
stefanpenner merged 4 commits intoember-cli:masterfrom
kellyselden:embroider-update
Sep 9, 2021
Merged

update embroider#9597
stefanpenner merged 4 commits intoember-cli:masterfrom
kellyselden:embroider-update

Conversation

@kellyselden
Copy link
Copy Markdown
Member

No description provided.

@stefanpenner
Copy link
Copy Markdown
Contributor

I believe the failure is webpack 5 + ember-data creating a cycle. The debugging is tedious because our add-on tests use JSDom, and JSDom basically throws uncaught exceptions evaluated in itself in a way that isn't much fun for debugging.

I am currently going to attempt to confirm my hypothesis, in hopes of identifying a path forward.

@stefanpenner
Copy link
Copy Markdown
Contributor

stefanpenner commented Aug 26, 2021

diagnosis: embroider-build/embroider#946
proposed fix: warp-drive-data/warp-drive#7680


It does seem like embroider@0.43.3 fixes the problem in a different way. But it will be good for ember-data to work properly anyways...

Let me see whats-up here.

@stefanpenner
Copy link
Copy Markdown
Contributor

rebasing this now

@stefanpenner
Copy link
Copy Markdown
Contributor

stefanpenner commented Aug 27, 2021

I believe the fix for the inflector & embroider issue is actually with embroiders inflector compatibility shim -> embroider-build/embroider#948 pending review from @rwjblue or @ef4

* reject promise if the evalScripts throws an error
* use AssertError to try and provide better test failures
* unfortunately errors thrown in JSDoms vmcontext are not provided with a stack. There may be an alternative solution to this, but I am not sure yet. Anyways, this is already better then before.
@stefanpenner
Copy link
Copy Markdown
Contributor

oh boy, the embroider tests passed!!!!! 🦖

@stefanpenner stefanpenner enabled auto-merge (squash) September 9, 2021 03:49
@stefanpenner stefanpenner merged commit 9c8ca9e into ember-cli:master Sep 9, 2021
@kellyselden kellyselden deleted the embroider-update branch September 9, 2021 17:50
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.

2 participants