Skip to content
This repository has been archived by the owner on Jan 12, 2024. It is now read-only.

incompatible typescript version - fixes #379 #380

Closed
wants to merge 2 commits into from
Closed

incompatible typescript version - fixes #379 #380

wants to merge 2 commits into from

Conversation

silkroadnomad
Copy link

@silkroadnomad silkroadnomad commented Jun 25, 2022

I removed the ^ from the typescript package because 4.7.4 seems incompatible with the Angular compiler.
This prevented a successful test run with yarn run test

Tests are now all running through.

@silkroadnomad silkroadnomad changed the title incompatible type script version - fixes #379 incompatible typescript version - fixes #379 Jun 25, 2022
@achingbrain
Copy link
Contributor

achingbrain commented Jun 27, 2022

CI is for the Angular example passes without these changes so they may not be necessary?

What is the error message you see without them? - you added the error message to #379 👍

Do you need to update your local lockfile or similar?

@silkroadnomad
Copy link
Author

silkroadnomad commented Jun 28, 2022

Good point! The tests are running through. Tried it on another machine now and got the same error. Maybe the modules are cashed on the Github-CI or so? Because this seems to be a new symptom. 4.7.4 was released only 11 days ago. Is CI cashing older modules of the same minor version?

@achingbrain
Copy link
Contributor

There's no caching in CI, it just runs a straight npm i.

@silkroadnomad silkroadnomad closed this by deleting the head repository Oct 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants