search

andrewdavey / cassette

Manages .NET web application assets (scripts, css and templates)
http://getcassette.net
MIT License
536 stars 144 forks source link

JSLint JavaScript code #7

Open andrewdavey opened 13 years ago

andrewdavey commented 13 years ago

Add configuration setting to run JSLint on JavaScript because no one hates your code quite as much as Crockford does!

Probably run the lint when the script modules are being built. Throw an exception on failure?

bakesteve commented 12 years ago

my 2p on this feature (as its something I'd definitely use) For failures I'd add some form of file logger (with a link from the dashboard?) scenario I can see being most useful would be to combine this into some form of build step that we could then hook into our CI (jenkins, in our case), so having the results feed into an xml file in xUnit form would be grand. An added bonus might be a command line flag to JUST do lint

Other part would be config to control the options (and lint engine - jsLint or jsHint) for that - the options (and maybe code to run lint or hint) from the jsLint visual studio add in should give an exhaustive list: http://visualstudiogallery.msdn.microsoft.com/961e6734-cd3a-4afb-a121-4541742b912e

Oh - and for now, the plugin above might make this less of a priority, as you can set it up to run on build or on save, so maybe this covers most peoples scenarios?

andrewdavey commented 12 years ago

Thanks for the suggestions. Displaying lint errors in the dashboard would be very handy during development.

I'm thinking about using a file in the project root to store the lint settings.

bakesteve commented 12 years ago

Sounds good. And a seperate configuration file definitely better than using web.config etc On Jan 4, 2012 4:08 PM, "Andrew Davey" < reply@reply.github.com> wrote:

Thanks for the suggestions. Displaying lint errors in the dashboard would be very handy during development.

I'm thinking about using a file in the project root to store the lint settings.

Reply to this email directly or view it on GitHub: https://github.com/andrewdavey/cassette/issues/7#issuecomment-3356513