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

npm run build breaks #39

Closed
oteinone opened this issue Apr 6, 2021 · 3 comments
Closed

npm run build breaks #39

oteinone opened this issue Apr 6, 2021 · 3 comments

Comments

@oteinone
Copy link

oteinone commented Apr 6, 2021

Please provide us with the following information:

This issue is for a: (mark with an x)

- [x] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

  1. Clone Repository
  2. run npm run build
  3. Update typescript version to e.g. 3.8.3 to get rid of issues with jest-diff
  4. run npm run build again

Any log messages given by the failure

1: .../node_modules/jest-diff/build/diffLines.d.ts:8:13 - error TS1005: '=' expected
2: TypeScript error in .../communication-services-web-calling-hero/Calling/ClientApp/src/containers/GroupCall.ts(39,26):
Property 'streams' does not exist on type 'StreamsState'.
3: TypeScript error in .../communication-services-web-calling-hero/Calling/ClientApp/src/core/RemoteStreamSelector.ts(63,20):
Member 'dominantParticipantsCount' implicitly has an 'any' type.

Expected/desired behavior

The build is successful

OS and Version?

Windows 10

Versions

node v15.13.0
npm 7.7.6

Mention any other details that might be useful

After updating typescript version to 3.8.3 and fixing the issues in groupcall.ts and remotestreamselector.ts everything seems to work normally


Thanks! We'll be in touch soon.

@alkwa-msft
Copy link
Contributor

Thanks for finding this I will be fixing this and pushing the fix up so others won't run into this!

@oteinone
Copy link
Author

oteinone commented Apr 6, 2021

Great! I would have made a pull request but I had a bunch of other stuff in the repo, including my connection string ...

@alkwa-msft
Copy link
Contributor

thats all good!

should be resolved with abe2a0b

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

No branches or pull requests

2 participants