-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Consider a temporary node_modules installation for dtslint #4079
Comments
This was referenced Sep 3, 2018
cartant
added a commit
to cartant/rxjs
that referenced
this issue
Sep 3, 2018
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Bug Report
Current Behavior
There is a problem that sometimes occurs with
dtslint
. In particular, it usually occurs on Travis and usually occurs around the time of a TypeScript release (AFAICT).When
dtslint
runs, it installs a bunch of TypeScript versions underneath its own directory innode_modules
. It seems that using a lock file sometimes cause errors like this:It seems that the error is effected because whatever
dtslint
downloads expects to be run in the presence of dependencies that are up-to-date and not pinned by a lock file.My workaround for this is to update the lock file using:
And to then push the updated lock file in a commit. I've used this procedure to fix the error on several occasions.
Possible Solution
Given that
dtslint
downloads modules when it runs and doesn't expect its dependencies to be pinned, it might be better to use a separate, temporarynode_modules
installation fordtslint
itself.The text was updated successfully, but these errors were encountered: