-
Notifications
You must be signed in to change notification settings - Fork 105
Required to point to the tslint-cli.js file when running on windows #170
Comments
I had the same issue. I tried to run lint with the produced command (-X switch). TSLint complained about a missing file or directory:
I run it again and changed the --out parameter to something like
It is working also with the solution described by @kingsleymasters . |
So @habbo656 I think your issue is different, in that it looks like maybe the user account running the scanner doesn't have permissions to create folders and write to the filesystem? I've seen similar on TFS installs before - the plugin relies on the scanner having created the temporary folder for it and that it can write to it so I'd check permissioning first. @kingsleymasters I'm not sure why you're seeing that unless there's been a change to how |
I forgot to mention that I'm using maven-sonar-plugin. I don't think that this are permission issues. |
Does everything up to the .sonartmp path component exist on disk? |
Yes, I run |
I have a reproduction for you I am using node 6.10.2 and tslint 5.5.0. I've taken these command lines from the sonarqube logs. Succeeds
Fails
I think it fails as its trying to call tslint using node, but on windows tslint is a batch file, workaround works perfectly so maybe just needs documenting... |
In order to get tslint working I have had to set the path directly to the javascript file for the CLI
It looks like it maybe should call tslint as an executable on windows using the .cmd wrapper instead of prefixing it with node.
Updating the config to this resolved the issue for me
sonar.ts.tslint.path=node_modules\tslint\lib\tslint-cli.js
Otherwise just fails silently, it looks like it is sucessful, but no issues are imported
Running SonarQube Version 6.5 (build 27846) and v1.1.0 of the SonarTsPlugin
The text was updated successfully, but these errors were encountered: