this became auto-fixable in eslint v3, it was previously applied to only part of the code base

this became auto-fixable in eslint v3, it was previously fixed manually in part of the code base

this rule was added to the shared eslint config[1] and was automatically fixed

[1]: https:/elastic/eslint-config-kibana

this became auto-fixable in eslint v3

this was added to the shared eslint config[1] a while ago and is not autofixable. Instead, a collection of custom jscodeshift transforms[2] were used to fix certain violation formats, and the rest was fixed manually.

[1]: https:/elastic/eslint-config-kibana
[2]: https:/kreeware/kibana-jscodeshift/tree/master/transforms

The import statements produced by jscodeshift do not put spaces between the properties and the curly-braces used in destructuring import statements. This is different from the style they are usually written in, so I enabled this rule to fix them and ensure we keep with the current style.

Several of the timer stubs created a reference to a clock variable, but never used that variable, so they were causing no-unused-vars errors. In the process of fixing those errors, some of those must have been removed, which caused some test failures.

When the tests for the api were written, they ended up using intern. This is not desirable for a number of reasons, but in an effort to remove all remaining `require()` calls I decided to give this migration a shot. Moving the tests to mocha was actually trivial.