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

Build error with embroider #349

Closed
enspandi opened this issue Dec 3, 2021 · 2 comments · Fixed by #350
Closed

Build error with embroider #349

enspandi opened this issue Dec 3, 2021 · 2 comments · Fixed by #350

Comments

@enspandi
Copy link

enspandi commented Dec 3, 2021

Trying to set up embroider - I ran into this build issue:

Build Error (PackagerRunner) in ../../node_modules/@ember/test-waiters/index.ts

Module parse failed: Unexpected token (1:7)
File was processed with these loaders:
 * ...node_modules/@embroider/babel-loader-8/index.js
You may need an additional loader to handle the result of these loaders.
> export type { WaiterName, Token, Primitive, Waiter, TestWaiter, TestWaiterDebugInfo, PendingWaiterState } from './types';

Is this maybe because of the main field pointing to the index.ts instead of index.js?

https://github.com/emberjs/ember-test-waiters/blob/master/package.json#L14

https://www.typescriptlang.org/docs/handbook/declaration-files/publishing.html#including-declarations-in-your-npm-package

@aaronucsd
Copy link

I'm getting the same error

ERROR in ../../node_modules/@ember/test-helpers/-internal/debug-info.js 19:0-60
Module not found: Error: Can't resolve '@ember/test-waiters' in '$TMPDIR/embroider/bfc6a7/node_modules/@ember/test-helpers/-internal'
 @ ./assets/test.js 46:9-82

@scalvert
Copy link
Collaborator

@enspandi ya I don't think it's supposed to point to a .ts file. I can look at fixing this.

@aaronucsd that error isn't the same as the one above. That looks like a problem with your local topology in your node_modules. I'd blow it away and install again.

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 a pull request may close this issue.

3 participants