-
Notifications
You must be signed in to change notification settings - Fork 137
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
broccoliBuilderErrorStack … Can't resolve '../../test-waiters' #778
Comments
is anyone looking into this? |
@rwjblue mentioned an issue recently with the way that If you use something like yarn resolutions to force your app to have a single copy of |
my addon doesn't have ember/test-waiters in package.json from yarn.lock, I see there are ^3.0.0 and ^2.4.3 which version should my addon try to resolve to? |
|
@angelayanpan I can confirm to add a resolution entry in the root "resolutions": {
"@ember/test-waiters": "3.0.1"
} |
thanks! but this is not a permanent solution right? |
I think this is fixed by #1064 |
when do you plan to release the above PR? |
It's already released I think: https://github.com/embroider-build/embroider/blob/master/CHANGELOG.md#v0500-2022-01-08 |
ok, so I have pulled in I still see "can't resolve test-waiters" in my project. I added the resolution back. and don't see that specific error anymore. |
Yes, the PR I mentioned above is released on 0.50. This issue may be a combination of people reporting two different problems that interact, and only one of them is fixed. @angelayanpan is your stack trace still exactly the same as the ones above? Is the place that's failing to find test-waiters using it via the old import path ( |
no occurances of "ember-test-waiters" in our project when I did a search.
and more.... (21 errors in total) |
@angelayanpan - This is after updating to @embroider/* 0.50.0? |
yes @rwjblue |
Similar situation here. Added @ember/test-waiters to a project with @embroider/* 0.50.2 and getting the following:
The import statement is |
you probably need add to the |
Thanks @angelayanpan but this particular project doesn't use Yarn so that's not an option, unless I've missed some recent NPM feature releases? Edit: so it looks like I did miss something and "overrides" were added to NPM in 8.3.0. Will give that a try now 👍🏻 Edit: added an npm override to my package.json but it did not fix the issue for my project. |
@roomman I had the same issue, build failed with adding |
I can't tell if this is related to this issue, but I thought I'd give embroider a try and I get a build failure on step 1 ("First make it work with no options. This is the mode that supports maximum backward compatibility."):
- broccoliBuilderErrorStack: ModuleNotFoundError: Module not found: Error: Can't resolve '../../test-waiters' in '$TMPDIR/embroider/6d8b07/node_modules/@ember/test-helpers/-internal/debug-info.js'
This is with Ember 3.23.1 and Embroider 0.40.0
The text was updated successfully, but these errors were encountered: