Linting sensor does not work with tslint > 5.2.0 on multi-module project

Pablissimo / SonarTsPlugin

SonarQube plugin for TypeScript files

MIT License

185 stars 106 forks source link

Linting sensor does not work with tslint > 5.2.0 on multi-module project #147

Open andreasgebauer opened 7 years ago

andreasgebauer commented 7 years ago

I get the following log message when trying to analyze a Maven multi-module project with tslint greater than 5.2.0:

[WARNING] TsLint reported issues against a file that isn't in the analysis set - will be ignored: <maven-module-name>/<module-relative-path-to-ts-file>

Seems like the working directory should be set to the current module in the Command instance created in TsLintExecutorImpl.getBaseCommand. This will definitely fix my problem.

Pablissimo commented 7 years ago

Thanks, will take a look at the PR - it might also address #149 actually...

OneCyrus commented 7 years ago

same problem here. can we get an updated release with PR #148?

Pablissimo commented 7 years ago

Trying to find time, work + impending fatherhood consuming most of it - will try and sneak a merge and pre-release in this weekend but if you're in a rush then building yourself's relatively straightforward

JRChampion commented 6 years ago

I am also seeing this issue. Is there any chance we can get a new release with PR #148?

andreasgebauer commented 6 years ago

Just discovered https://github.com/SonarSource/SonarTS. This seems to be more active.