search

JoshuaKGoldberg / TSLint.MSBuild

[Deprecated] An MSBuild task for running TSLint.
https://www.nuget.org/packages/TSLint.MSBuild
MIT License
15 stars 13 forks source link

How to use TSLintCli property? #67

Closed brettjacobson closed 7 years ago

brettjacobson commented 7 years ago

I've tried every permutation I could on the TSLintCli property to try and make it use the tslint that is installed in our node_modules folder, with no luck. What do you think the magic syntax should be to force it to use tslint acquired through npm instead of using a NuGet package to download TSLint into the packages folder?

brettjacobson commented 7 years ago

Never mind looks like I had something else stupid broken that was completely preventing it from running.

node_modules/tslint/bin/tslint

works fine. Would it be a good idea to make this the highest priority check, before digging through the Packages folder?

JoshuaKGoldberg commented 7 years ago

Sure, that seems reasonable. PR?

brettjacobson commented 7 years ago

I'm naive when it comes to git and PRs. I'll see if I can figure out how to do them this week.

But it looks like the magic to add to the .targets is this (before the others): <TSLintPotentialCli Include="$(ProjectDir)node_modules/tslint/bin/tslint" />