Skip to content
This repository has been archived by the owner on Jul 8, 2019. It is now read-only.

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

Open
andreasgebauer opened this issue Jun 8, 2017 · 5 comments

Comments

@andreasgebauer
Copy link

andreasgebauer commented Jun 8, 2017

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.

andreasgebauer pushed a commit to andreasgebauer/SonarTsPlugin that referenced this issue Jun 8, 2017
andreasgebauer pushed a commit to andreasgebauer/SonarTsPlugin that referenced this issue Jun 8, 2017
@Pablissimo
Copy link
Owner

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

@OneCyrus
Copy link

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

@Pablissimo
Copy link
Owner

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

quidio pushed a commit to wayoos/SonarTsPlugin that referenced this issue Aug 23, 2017
@JRChampion
Copy link

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

@andreasgebauer
Copy link
Author

andreasgebauer commented Dec 5, 2017

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants