867283 - Switch to Grunt for all Linting

Status: RESOLVED FIXED

(Webmaker Graveyard :: Popcorn Maker, defect)

Comment 1

[Matthew Schranz [:mjschranz]]

Silly me. We've wanted to do this for a bit of time now and we are trying to unify the tech we use across all of our applications.

We use LESS already so that's taken care of, but we don't have Grunt implemented in any form in this application yet.

Comment 2

[Matthew Schranz [:mjschranz]]

One additional thing we need to do here is write a Grunt plugin for https://github.com/mozilla/html5-lint which we use for our HTML linting. Options exist for linting js and LESS.

Comment 3

[Matthew Schranz [:mjschranz]]

Apparently :humph already took a stab at getting this sort of grunt plugin working.

Ali, I would take this code and make it a module that any repo can include so we can use this cross project. See https://github.com/mozilla/webmaker.org/pull/14.

Comment 5

[Ali Al Dallal [:alicoding]]

I have :humph codes published to https://npmjs.org/package/grunt-lint5 and not sure if we need to have this transfer first or?

Comment 6

[Pomax]

do this concurrently? "all linting" I imagine means CSS, HTML and JS linting. You could block this issue on those three subtasks (some of which we obviously already do)

Comment 7

[Matthew Schranz [:mjschranz]]

Attachment: https://github.com/mozilla/popcorn.webmaker.org/pull/307

Weeeee

For now this removes our linting of HTML files and that's going to be added in a follow up (see current blocks list). Besides, it's only linting the layout files we use that aren't templates.

Apparently our old linting was really bad and didn't catch a whooole lot of stuff. Obviously minor since nothing's gone wrong but still. It was kind of busted. This also allows us to enforce some things like quote style.

Comment 8

[Scott [:thecount] Downe]

Comment on attachment 829806 [details] [review]
https://github.com/mozilla/popcorn.webmaker.org/pull/307

Fix one comment and land it.

Comment 9

[[github robot]]

Commit pushed to master at https://github.com/mozilla/popcorn.webmaker.org

https://github.com/mozilla/popcorn.webmaker.org/commit/8833fa81067b478752cc441190e794c4cfd0bcb8
Fix Bug 867283 - Add grunt support to Popcorn Maker. Fix a whole crap tonne of hidden lint errors and double quotes vs single quotes