{"action":"opened","issue":{"url":"https://api.github.com/repos/phetsims/chipper/issues/97","labels_url":"https://api.github.com/repos/phetsims/chipper/issues/97/labels{/name}","comments_url":"https://api.github.com/repos/phetsims/chipper/issues/97/comments","events_url":"https://api.github.com/repos/phetsims/chipper/issues/97/events","html_url":"https://github.com/phetsims/chipper/issues/97","id":57387795,"number":97,"title":"chipper/grunt/*.js fails lint","user":{"login":"pixelzoom","id":3046552,"avatar_url":"https://avatars.githubusercontent.com/u/3046552?v=3","gravatar_id":"","url":"https://api.github.com/users/pixelzoom","html_url":"https://github.com/pixelzoom","followers_url":"https://api.github.com/users/pixelzoom/followers","following_url":"https://api.github.com/users/pixelzoom/following{/other_user}","gists_url":"https://api.github.com/users/pixelzoom/gists{/gist_id}","starred_url":"https://api.github.com/users/pixelzoom/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pixelzoom/subscriptions","organizations_url":"https://api.github.com/users/pixelzoom/orgs","repos_url":"https://api.github.com/users/pixelzoom/repos","events_url":"https://api.github.com/users/pixelzoom/events{/privacy}","received_events_url":"https://api.github.com/users/pixelzoom/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/phetsims/chipper/labels/bug","name":"bug","color":"fc2929"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-02-11T22:30:22Z","updated_at":"2015-02-11T22:30:22Z","closed_at":null,"body":"```js\r\nLinting grunt/Gruntfile.js ...ERROR\r\n[L18:C5] W079: Redefinition of '_'.\r\nvar _ = require( '../../sherpa/lodash-2.4.1.min' );\r\nLinting grunt/Gruntfile.js ...ERROR\r\n[L30:C3] E007: Missing \"use strict\" statement.\r\n  function trimWhitespace( str ) {\r\nLinting grunt/Gruntfile.js ...ERROR\r\n[L57:C46] W064: Missing 'new' prefix when invoking a constructor.\r\n    return 'data:' + mimeType + ';base64,' + Buffer( fs.readFileSync( filename ) ).toString( 'base64' );\r\nLinting grunt/Gruntfile.js ...ERROR\r\n[L193:C33] W069: ['en'] is better written in dot notation.\r\n        var fallback = stringMap[ 'en' ];\r\nLinting grunt/Gruntfile.js ...ERROR\r\n[L278:C46] W069: ['en'] is better written in dot notation.\r\n    var fallbackStrings = global_phet_strings[ 'en' ];\r\nLinting grunt/Gruntfile.js ...ERROR\r\n[L371:C65] W065: Missing radix parameter.\r\n    var number = parseInt( pkg.version.substring( lastDot + 1 ) );\r\nLinting grunt/Gruntfile.js ...ERROR\r\n[L443:C24] W069: ['en'] is better written in dot notation.\r\n    global.phet.strings[ 'en' ] = {};//may overwrite above\r\nLinting grunt/Gruntfile.js ...ERROR\r\n[L614:C70] W038: 'strings' used out of scope.\r\n        iframeTestHtml = stringReplace( iframeTestHtml, 'SIM_TITLE', strings[ titleKey ] + ' ' + pkg.version + ' iframe test' );\r\nLinting grunt/Gruntfile.js ...ERROR\r\n[L614:C79] W038: 'titleKey' used out of scope.\r\n        iframeTestHtml = stringReplace( iframeTestHtml, 'SIM_TITLE', strings[ titleKey ] + ' ' + pkg.version + ' iframe test' );\r\nLinting grunt/Gruntfile.js ...ERROR\r\n[L623:C23] W003: 'stringMap' was used before it was defined.\r\n        var stringMap = 'build/' + pkg.name + '_string-map.json';\r\nLinting grunt/Gruntfile.js ...ERROR\r\n[L57:C46] W117: 'Buffer' is not defined.\r\n    return 'data:' + mimeType + ';base64,' + Buffer( fs.readFileSync( filename ) ).toString( 'base64' );\r\nLinting grunt/Gruntfile.js ...ERROR\r\n[L79:C16] W098: 'requirejs' is defined but never used.\r\n  var requirejs = require( '../../' + pkg.name + '/node_modules/requirejs' ); // TODO: not currently used, figure out how to include almond correctly?\r\nLinting grunt/createSim.js ...ERROR\r\n[L109:C15] W117: 'fs' is not defined.\r\n        if ( !fs.existsSync( destPath ) || overwrite ) {\r\nLinting grunt/createSim.js ...ERROR\r\n[L2:C11] W098: 'assert' is defined but never used.\r\nvar assert = require( 'assert' );\r\nLinting grunt/pullAll.js ...ERROR\r\n[L3:C11] W098: 'assert' is defined but never used.\r\nvar assert = require( 'assert' );\r\n```"}}