search

SamVerschueren / clinton

Project style linter
MIT License
122 stars 9 forks source link

Undefined line:column in some projects #47

Closed sindresorhus closed 8 years ago

sindresorhus commented 8 years ago 
ansi-regex

  package.json:undefined:undefined
  ✖  Specify AVA config in package.json instead of passing it through via the CLI.  ava

  1  error
anybar

  package.json:undefined:undefined
  ✖  Property dependencies should occur before property devDependencies.  pkg-property-order

  1  error
SamVerschueren commented 8 years ago

Will look into it tomorrow.

SamVerschueren commented 8 years ago

Oh ok, this is added by eslint-formatter-pretty. Why are these actually added and made invisible?

sindresorhus commented 8 years ago

Oh, right. It's so you can just click the title in iTerm (and other terminals supporting that) and have it open in your editor on the correct line. You need to add a conditional there too regarding the missing line:column.

SamVerschueren commented 8 years ago

Wow, very nice feature :). Will do a PR this evening.

SamVerschueren commented 8 years ago

Should be fixed after reinstalling because of eslint-formatter-pretty@0.3.1