search

cfjedimaster / brackets-jshint

Adds JSHint support to Brackets
MIT License
131 stars 41 forks source link

Add "unused" check on declared parameters that are unused. #67

Closed vajnorcan closed 9 years ago

vajnorcan commented 9 years ago

The jsHint toggle panel could contain two tabs, one for the "problems", and the other for "unused".

cfjedimaster commented 9 years ago

My extension uses the built in Brackets linting API so I can't really change the UI unless I completely change how the extension works.

On Tue, Dec 16, 2014 at 12:44 AM, vajnorcan notifications@github.com wrote:

The jsHint toggle panel could contain two tabs, one for the "problems", and the other for "unused".

— Reply to this email directly or view it on GitHub https://github.com/cfjedimaster/brackets-jshint/issues/67.

Raymond Camden, Web Developer for Adobe

Email : raymondcamden@gmail.com Blog : www.raymondcamden.com Twitter: raymondcamden

vajnorcan commented 9 years ago

Fair enough. Is there any chance to include the "unused" parameters in the "JSHint Problems" tab ? I mean the function parameters that are not used at all. Thanks. I'm using this plugin now quite heavily, great job.

cfjedimaster commented 9 years ago

Can't you control that with .jshintrc?

On Tue, Dec 16, 2014 at 5:25 PM, vajnorcan notifications@github.com wrote:

Fair enough. Is there any chance to include the "unused" parameters in the "JSHint Problems" tab ? Thanks. I'm using this plugin now quite heavily, great job.

— Reply to this email directly or view it on GitHub https://github.com/cfjedimaster/brackets-jshint/issues/67#issuecomment-67252071 .

Raymond Camden, Web Developer for Adobe

Email : raymondcamden@gmail.com Blog : www.raymondcamden.com Twitter: raymondcamden

vajnorcan commented 9 years ago

mea culpa. had the "unused" parameters set to "vars". Changed to true or "strict" helped.