angular/angular.js — pull request #10088 by hzoo (2014-11-17 04:39:54+00:00)
Title: Add jscs jsdoc

WIP

Don't know if we still want to add jsdoc checks but if we do... [jscs-jsdoc](https://github.com/jscs-dev/jscs-jsdoc).
Added `jscs-jsdoc`, updated `grunt-jscs` to `1.8.1` and `jscs` to `1.7.3` (no `1.8.0` yet).

Ngdoc is a bit different so not sure if it will work consistently.

Not sure if I should remove the fsevents from `npm-shrinkwrap.json`.

I did
`npm install jscs-jsdoc`
`npm shrinkwrap --dev`
`./scripts/clean-shrinkwrap.js`

Adding jscs rules:
`checkParamNames` to match the docs parameter names.

----
Could also standardize parameter names like `expression` vs `expr` that are used a lot.
Lots of trivial style decisions haha.

Also had a problem with `src/ngMessages/messages.js` line 349
where jscs had a problem with
`* @param {string} ngMessage a string value corresponding to the message key.`
since the parameter is the parameter of the directive and not a function so just did
`// jscs:disable jsDoc` for now.

Not sure why it didn't have a problem with directive `ngMessages` but had one with `ngMessage` - maybe have to check some of these manually.

can still add rule: `requireParamTypes` to make sure there is a {TYPE}

cc: @caitp