Removed a default eslint rule

Corrected spelling mistakes
Reverted unrelated changes

Author: Jeroen Meeus

index.js

@@ -288,11 +288,9 @@ const publicApi = {
      * than the DefinePlugin:
      *
      *      const Encore = require('@symfony/webpack-encore');
-     *      const PluginPriorities =
-     * require('@symfony/webpack-encore/lib/plugins/plugin-priorities.js');
+     *      const PluginPriorities = require('@symfony/webpack-encore/lib/plugins/plugin-priorities.js');
      *
-     *      Encore.addPlugin(new MyWebpackPlugin(),
-     * PluginPriorities.DefinePlugin);
+     *      Encore.addPlugin(new MyWebpackPlugin(), PluginPriorities.DefinePlugin);
      *
      * @param {string} plugin
      * @param {number} priority
@@ -637,23 +635,23 @@ const publicApi = {
      * https://github.com/MoOx/eslint-loader
      *
      *     // enables the eslint loaded using the default eslint configuration.
-     *     Encore.enableEslint();
+     *     Encore.enableEslintLoader();
      *
      *     // Optionally, you can pass in the configuration eslint should extend.
-     *     Encore.enableEslint('airbnb');
+     *     Encore.enableEslintLoader('airbnb');
      *
      *     // You can also pass in an object of options
      *     // that will be passed on to the eslint-loader
-     *     Encore.enableEslint({
+     *     Encore.enableEslintLoader({
      *         extends: 'airbnb',
                emitWarning: false
      *     });
      *
-     *     // For a mroe advanced usage you can pass in a callback
+     *     // For a more advanced usage you can pass in a callback
      *     // https://github.com/MoOx/eslint-loader#options
-     *     Encore.enableEslint((options) => {
+     *     Encore.enableEslintLoader((options) => {
      *          options.extends = 'airbnb';
-     *          options.emitWarning = fasle;
+     *          options.emitWarning = false;
      *     });
      *
      * @param {string|object|function} eslintLoaderOptionsOrCallback

lib/features.js

@@ -67,7 +67,7 @@ const features = {
         method: 'enableEslintLoader()',
         // eslint is needed so the end-user can do things
         packages: ['eslint', 'eslint-loader', 'babel-eslint', 'eslint-plugin-import'],
-        description: 'load VUE files'
+        description: 'Enable ESLint checks'
     },
     notifier: {
         method: 'enableBuildNotifications()',

lib/loaders/eslint.js

@@ -22,9 +22,6 @@ module.exports = {
         const eslintLoaderOptions = {
             parser: 'babel-eslint',
             emitWarning: true,
-            rules: {
-                'linebreak-style': 'off'
-            },
             'import/resolver': {
                 webpack: {
                     config: 'webpack.config.js'

test/loaders/eslint.js

@@ -28,7 +28,7 @@ describe('loaders/eslint', () => {
         config.enableEslintLoader();
         const actualOptions = eslintLoader.getOptions(config);
 
-        expect(Object.keys(actualOptions)).to.have.lengthOf(4);
+        expect(Object.keys(actualOptions)).to.have.lengthOf(3);
     });
 
     it('getOptions() with extra options', () => {
@@ -39,7 +39,7 @@ describe('loaders/eslint', () => {
 
         const actualOptions = eslintLoader.getOptions(config);
 
-        expect(Object.keys(actualOptions)).to.have.lengthOf(5);
+        expect(Object.keys(actualOptions)).to.have.lengthOf(4);
         expect(actualOptions.extends).to.equal('airbnb');
     });
 
@@ -51,7 +51,7 @@ describe('loaders/eslint', () => {
 
         const actualOptions = eslintLoader.getOptions(config);
 
-        expect(Object.keys(actualOptions)).to.have.lengthOf(4);
+        expect(Object.keys(actualOptions)).to.have.lengthOf(3);
         expect(actualOptions.emitWarning).to.equal(false);
     });
 });