visionmedia/should.js — pull request #55 by gabrielfalcao (2012-03-29 04:50:30+00:00)
Title: Proposing a fallback of `.instanceof` as `.instanceOf`

I've checked #47 and I totally agree with the fact that `.instanceof` is ok to be used even though it's a reserved keyword.

But it turns out that I have a particular example of when that fallback would help

1. Emacs running jshint

![http://f.cl.ly/items/040b2x1A3r060R1a011P/Screen%20Shot%202012-03-29%20at%2012.44.12%20AM.png](http://f.cl.ly/items/040b2x1A3r060R1a011P/Screen%20Shot%202012-03-29%20at%2012.44.12%20AM.png)

2. jshint output

![http://f.cl.ly/items/1c0N1s0V3M0i3G1R3i01/Screen%20Shot%202012-03-29%20at%2012.44.51%20AM.png](http://f.cl.ly/items/1c0N1s0V3M0i3G1R3i01/Screen%20Shot%202012-03-29%20at%2012.44.51%20AM.png)

As a side note, I'm sorry if the implementation is not optimal. I love javascript and I'm in the learning process, so I'm not very fluent yet.

But at least I tried to add appropriate test coverage :)

Last but not least, I was also going to propose a change in package.json in order to support the conventional `npm test`, but when I tried that mocha did not produce any output. I was wondering what's the reason.

PS.: an honor to send a patch to you :raised_hands: