Implement new eslint rule object-shorthand

[AlexKVal]

• When the issue object-shorthand rule fails with quoted properties eslint/eslint#2374 is resolved,
• eslint@0.21.0 version with fix is released.
• Implement new eslint rule object-shorthand.
• Wait for issue Rule 'object-shorthand' with es7.objectRestSpread eslint/eslint#2486 with es7.objectRestSpread is resolved
• Wait / find a solution for object-shorthand with es7.objectRestSpread
• Remove all disabling comments

1 NavItem.js#L32
2 Input.js#L181
3 Button.js#L34

ref: http://eslint.org/docs/rules/object-shorthand

[dozoisch]

2374 seems to be fixed now!

[AlexKVal]

Thank you, Hugo ✨
I've updated the issue progress.
Now we wait for new version release containing this fix.

[AlexKVal]

I've added
Wait for issue eslint/eslint#2486 with es7.objectRestSpread is resolved

[AlexKVal]

As it was expected es7.objectRestSpread rule is experimental and eslint doesn't support es7 yet.
Maybe babel-eslint monkeypatching.
It needs additional research.

[AlexKVal]

I've done some research on babel-eslint.

The babel-eslint "monkeypatching" is not patching any rules.

Therefore it seems the only solution is to comment out those lines for now.
(or write standalone plugin with this rule working with es7, as Ilya Volodin suggests)

As for this PR:
Maybe tag it as needs dependency update ?

[mtscout6]

Have you submitted the issue with babel-eslint?

[AlexKVal]

It's useless.
babel-eslint is just kind of a sugar for AST. That's all.
It has no any connection with any rule evaluation at all.

ref: https://github.com/babel/babel-eslint/blob/master/index.js
that's all.

[mtscout6]

Ah, I gotcha 😕

[AlexKVal]

Resolved by #872
and #875