-
Notifications
You must be signed in to change notification settings - Fork 12k
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
Cannot find type definition file for 'chai'. #3071
Comments
Did you change the rxjs version? It should be beta12. |
Actually I did update that to newest one, because So how should users know which packages to update and which not? Although it seems to work just fine if I just remove |
@tarlepp It seems rxjs requires
I think chai is a test framework, so hope rxjs should not include the dependency in the released module. |
It is ReactiveX/rxjs#2112 (Which in turn apparently is a TypeScript compiler issue), so it is on the radar. Manually adding chai types as dev dependency as @tmtk75 says is an easy workaround for now. |
Angular has a fixed peerDep version of 5.0.0-beta.12. I wouldn't update until angular does as well. |
We believe we have fixed this in RxJS 5.0.0-rc3, which was just published. ReactiveX/rxjs#2120 Edit: we accidentally didn't include the fix in that release..so it's not fixed. Sorry folks! 😞 |
still getting error with RxJS 5.0.0-rc3 |
😢 |
just upgraded to rc3, still there. |
same issue here with rc3 |
The above comment |
@MarkPieszak |
Missed that, thank you! Perfect. @jmesa-sistel |
The solution of @tmtk75 worked for me on RxJS RC5 |
Yeah installing |
this should be easily fixed but adding the dependency worked |
* Build working, server runs * move devDependencies, heroku doesn't pick them up * trying different approach * fix missing types for rxjs dependencies angular/angular-cli#3071 * missing environment variable for api URL * Fix all api post/put calls that require user id
This got fixed with lastes rxjs, closing the ref issue ReactiveX/rxjs#2112. |
Yeah I can confirm that latest rxjs 5.0.0-rc.4 solves this, no need for |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
OS?
Ubuntu 16.04
Versions.
angular-cli: 1.0.0-beta.19-3
node: 6.7.0
os: linux x64
Repro steps.
npm install
npm start
The log given by the failure.
The text was updated successfully, but these errors were encountered: