Chore: Add metadata to existing rules - Batch 6 (refs #5417)

[vitorbal]

Chore: Add metadata to existing rules - Batch 6 of 7 (refs #5417)

[eslintbot]

LGTM

[mention-bot]

By analyzing the blame information on this pull request, we identified @alberto, @mysticatea and @IanVS to be potential reviewers

[pedrottimark]

Verified the meta.docs and meta.fixable properties compared to the README.md file.

[qfox]

Sorry, if I'm late, but diff should looks much better with fixes like:

exports.meta = {...};
exports.create = function(context) { ...

instead of assigning new object:
module.exports = { meta: ..., create: ... };.

[nzakas]

Yeah a bit late. We decided on this format a while back and have already converted a lot of rules to this format. So, we need to continue along this path.

[vitorbal]

Just one more batch left before the pain is done :)

[pedrottimark]

@zxqfox Yeah, no fun right now, but as a guess, assigning object literal to module.exports was considered important to emphasize the difference between new rule format and old asymmetrical rule format which assigned module.exports to the function and then the schema to a property. Compare:

• http://eslint.org/docs/developer-guide/working-with-rules-new
• http://eslint.org/docs/developer-guide/working-with-rules

[qfox]

@pedrottimark I understand that and strongly agree. Just afraid about git history for these files. We could do it saving indent. Nvm already ;-)