Re: [web-idl] Grunt/JSHint/Uglify

Team: With this commit
(https://github.com/extensibleweb/webidl.js/commit/987a2c789dbe188df8dcd50c
a84ed367c4f32b96) we now have testing infrastructure. I've included the
first test for smoke testing the build.

I do want to point out that at the moment there is boilerplate code there
to handle the RequireJS async nature of the lib. IMO this will be painful
for dev cycles later.

Please let me know if you find issues with running grunt.



On 1/9/13 12:33 PM, "Clint Hill" <clint.hill@gmail.com> wrote:

>Team: With this commit
>(https://github.com/extensibleweb/webidl.js/commit/2c219228dd70f82807bf800
>b
>9ca8fa043c1c8052) I've gotten the .jshintrc included in the options for
>JSHint.
>
>proto and evil are set.
>
>Please let me know if you run into troubles with grunt/jshint. I've
>encountered a bug in which I received a diff version of
>grunt-contrib-jshint than did Brian during npm install.
>
>
>
>On 1/9/13 10:44 AM, "Marcos Caceres" <w3c@marcosc.com> wrote:
>
>>
>>
>>
>>On Wednesday, January 9, 2013 at 5:38 PM, Brian Kardell wrote:
>>
>>> 
>>> 
>>> On Wed, Jan 9, 2013 at 12:28 PM, Marcos Caceres <w3c@marcosc.com
>>>(mailto:w3c@marcosc.com)> wrote:
>>> > 
>>> > 
>>> > 
>>> > On Wednesday, January 9, 2013 at 5:25 PM, Clint Hill wrote:
>>> > 
>>> > > Team: I was a bit quick to pull this trigger. Marcos merged the
>>>pull
>>> > > request, and I will change the config on master. I'm taking in the
>>> > > suggestion from Mat regarding the .jshintrc and will update
>>>everyone once
>>> > > I'm done.
>>> > 
>>> > 
>>> > Awesome, thanks for taking the lead here :) I'm sure there will be
>>>quite a few bumps as we get going - but it's great that we are moving on
>>>stuff.
>>> > 
>>> > --
>>> > Marcos Caceres
>>> 
>>> 
>>> Clint, 
>>> 
>>> Verify your grunt version et all... I am getting
>>> 
>>> brian$ grunt
>>> Running "jshint:all" (jshint) task
>>> Linting lib/interfaces/WebIDL.js...ERROR
>>> [L10:C43] The Function constructor is eval.
>>> interfaceObject = new Function(functionBody)(),
>>> [L48:C25] The '__proto__' property is deprecated.
>>> interfaceObject.__proto__ = Object.create({});
>>> 
>>> Warning: Task "jshint:all" failed. Use --force to continue.
>>> 
>>
>>We need allow proto and evil for jshint.
>>
>>-- 
>>Marcos Caceres
>>
>>
>>
>
>

Received on Wednesday, 9 January 2013 20:50:41 UTC