-
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
chore(test): add dtslint files and script #3944
Conversation
Generated by 🚫 dangerJS |
Hmm. I hadn't noticed that npm had made such a mess of the lock file. I will try to fix it up a little. |
though lockfile itself seems ok-change? |
I'm just going to re-apply the commits and do a single install of dtslint using the current lock file from master. I'm not going to manually mess with the lock file. The initial commits and install of dtslint were from some time ago and I had to re-install dtslint to get an updated dependency - as I'm creating this PR on a different computer - and I really don't think that the lock file should have as many changes in it as it appears to have. I'd like to be sure that what was in the |
@kwonoj I'm done messing with this. I don't know why npm does some of the things that it does. E.g. it appears to have changed Anyway, I'm at least confident that there wasn't any filth in |
Description:
This PR adds dtslint to the full-validation build. The dtslint-related files are in the
spec-dtslint
directory.The
type
tests for thezip
factory function and thezip
operator have been included so that they are tested using dtslint's expectations. The originaltype
tests have not been removed.Related issue (if exists): #3823