-
Notifications
You must be signed in to change notification settings - Fork 889
failing when running locally #1770
Comments
I couldn't reproduce this issue using the command line example. Does |
I couldn't find it in either. |
I have a new laptop as of today I'm going to try and recreate with the hope that I can't (but sadly with the believe that I will) |
why are files in |
|
yes, I understand. my question still stands |
Whoops, misread that! Likely the CLI just imports things at the top of the On Tue, Nov 22, 2016, 7:43 PM Adi Dahiya notifications@github.com wrote:
|
it's just part of the compilation process. When Node sees an import, it loads the file and compiles it. |
Neither local or global have the test directory as pointed out by @jkillian. The test file that is failing is If the |
what is in your lib directory? |
oh i mean in your tslint/lib directory? |
try:
|
I have now installed 4.0.2 (versus 4.0.1) globally and it too has the
In contrast, the local version (also 4.0.2 now) gives the same error message but rather than originating in the
|
try |
I installed with the above and now I do have the parse file: but I still get the same error when running |
May be a red herring but using the vs-code editor you can see below that there is a TSLint error: Looking at this error you find:
the source of this error appears to be a name change from |
You need to upgrade vscode-tslint due to an API change Try upgrading NPM. Since you're seeing something different when you install tslint using the tgz vs using Your global library may be corrupted somehow. Try clearing it out, verifying everything is uninstalled by running |
i removed all of my globally install npm files -- I then blow away my local environment too, and reinstall with yarn: As you can see I am still getting the same error. 😢 Then I start up vs-code (1.7.2) and get the following in the "tslint window":
Very odd indeed. Even odder is when I go to a typescript file I see this in the gutter: Version 2.1.1? I have version 2.0.10 installed globally and locally. |
I have recreated this on my brand new macbook pro (which had nothing installed to start).
Regardless if I choose "workspace version" or "bundled version", when restarting it fails to find tslint. 😢 |
maybe it's a yarn vs npm issue |
well it's possible but I did consider that and have tested with both |
I tested this on a new macbook pro which did not initially have npm, and it worked for me. |
I just ran into the same issue... was using YARN. Turns out that the issue for me was that I at some point ran Not sure what the best course of action is... either for yarn to remove |
@nsgundy thanks. Closing this ticket since there's nothing else for us to do here. |
Bug Report
2.0.10
Actual Behaviour
My lint runner is an npm script which defined as this:
If I run
tslint --force --format verbose "src/**/*.ts"
from the terminal it works without error:If, however, I run the npm lint script. It will run the locally install tslint and it fails with:
This can be reproduced by just running the following at the terminal:
Expected behavior
I would expect the local install of tslint to behave the same as the global install.
Final Note
In case it makes any difference, here are the deps in my project from
package.json
:The text was updated successfully, but these errors were encountered: