panuhorsmalahti / gulp-tslint

TypeScript linter plugin for Gulp
MIT License
118 stars 44 forks source link

linter always uses file.relative #67

Closed nojvek closed 8 years ago

nojvek commented 8 years ago

using something like this

gulp.src("foo/**/*.ts").pipe(tslint({formatter: "full"))

Currently gives

 [gulp-tslint] error (semicolon) hello/world.ts[82, 67]: Missing semicolon

I would expect the path to be either

foo/hello/world.ts

or

/Users/me/Documents/dev/foo/hello.world.ts

The ask is because I'm working on a tslint autofixer that can take the output of tslint or gulp-tslint and auto fix the errors.

Currently the gulp-tslint output is not very machine friendly since the paths it emits is missing info.

panuhorsmalahti commented 8 years ago

There was a formatter "full" which gives the full path, but this was removed today. For now, you would need to implement your own formatter to get the full path I think, not sure if tslint natively provides the full path to the file.

nojvek commented 8 years ago

Why was the full formatter removed though?

On Saturday, July 9, 2016, Panu Horsmalahti notifications@github.com wrote:

There was a formatter "full" which gives the full path, but this was removed today. For now, you would need to implement your own formatter to get the full path I think, not sure if tslint natively provides the full path to the file.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/panuhorsmalahti/gulp-tslint/issues/67#issuecomment-231550912, or mute the thread https://github.com/notifications/unsubscribe/AA-JVM7-LiHABvG-Mcag32j0yZp_rmNFks5qT_N9gaJpZM4JHk0m .

panuhorsmalahti commented 8 years ago

Because we removed support for gulp-tslint formatters, and only support tslint formatters.

panuhorsmalahti commented 8 years ago

Closing as won't fix.