Merge pull request #11998 from hasezoey/eslint-eollast

add eslint rule `eol-last` to change EOL to be consistent

.eslintrc.json

@@ -186,6 +186,7 @@
     "mocha-no-only/mocha-no-only": [
       "error"
     ],
-    "no-empty": "off"
+    "no-empty": "off",
+    "eol-last": "warn"
   }
-}
+}

lib/cast.js

@@ -374,4 +374,4 @@ function _cast(val, numbertype, context) {
       }
     }
   }
-}
+}

lib/cast/date.js

@@ -38,4 +38,4 @@ module.exports = function castDate(value) {
   }
 
   assert.ok(false);
-};
+};

lib/cast/decimal128.js

@@ -33,4 +33,4 @@ module.exports = function castDecimal128(value) {
   }
 
   assert.ok(false);
-};
+};

lib/cast/objectid.js

@@ -26,4 +26,4 @@ module.exports = function castObjectId(value) {
   }
 
   return new ObjectId(value);
-};
+};

lib/drivers/node-mongodb-native/index.js

@@ -9,4 +9,4 @@ exports.Collection = require('./collection');
 exports.Decimal128 = require('./decimal128');
 exports.ObjectId = require('./objectid');
 exports.ReadPreference = require('./ReadPreference');
-exports.getConnection = () => require('./connection');
+exports.getConnection = () => require('./connection');

lib/error/parallelValidate.js

@@ -28,4 +28,4 @@ Object.defineProperty(ParallelValidateError.prototype, 'name', {
  * exports
  */
 
-module.exports = ParallelValidateError;
+module.exports = ParallelValidateError;

lib/helpers/aggregate/prepareDiscriminatorPipeline.js

@@ -34,4 +34,4 @@ module.exports = function prepareDiscriminatorPipeline(pipeline, schema, prefix)
       originalPipeline.unshift({ $match: match });
     }
   }
-};
+};

lib/helpers/aggregate/stringifyFunctionOperators.js

@@ -47,4 +47,4 @@ function handleAccumulator(operator) {
       operator.$accumulator[key] = String(operator.$accumulator[key]);
     }
   }
-}
+}

lib/helpers/arrayDepth.js

@@ -30,4 +30,4 @@ function arrayDepth(arr) {
   res.max = res.max + 1;
 
   return res;
-}
+}

lib/helpers/cursor/eachAsync.js

@@ -189,4 +189,4 @@ function createEachAsyncMultiError(aggregatedErrors) {
   }
 
   return new EachAsyncMultiError(aggregatedErrors);
-}
+}

lib/helpers/discriminator/areDiscriminatorValuesEqual.js

@@ -13,4 +13,4 @@ module.exports = function areDiscriminatorValuesEqual(a, b) {
     return a.toString() === b.toString();
   }
   return false;
-};
+};

lib/helpers/discriminator/checkEmbeddedDiscriminatorKeyProjection.js

@@ -9,4 +9,4 @@ module.exports = function checkEmbeddedDiscriminatorKeyProjection(userProjection
       addedPaths[0] === _discriminatorKey) {
     selected.splice(selected.indexOf(_discriminatorKey), 1);
   }
-};
+};

lib/helpers/discriminator/getConstructor.js

@@ -22,4 +22,4 @@ module.exports = function getConstructor(Constructor, value) {
   }
 
   return Constructor;
-};
+};

lib/helpers/discriminator/getDiscriminatorByValue.js

@@ -24,4 +24,4 @@ module.exports = function getDiscriminatorByValue(discriminators, value) {
     }
   }
   return null;
-};
+};

lib/helpers/discriminator/getSchemaDiscriminatorByValue.js

@@ -23,4 +23,4 @@ module.exports = function getSchemaDiscriminatorByValue(schema, value) {
     }
   }
   return null;
-};
+};

lib/helpers/document/handleSpreadDoc.js

@@ -32,4 +32,4 @@ module.exports = function handleSpreadDoc(v, includeExtraKeys) {
   }
 
   return v;
-};
+};

lib/helpers/each.js

@@ -22,4 +22,4 @@ module.exports = function each(arr, cb, done) {
       }
     });
   }
-};
+};

lib/helpers/get.js

@@ -61,4 +61,4 @@ function getProperty(obj, prop) {
     return obj.get(prop);
   }
   return obj[prop];
-}
+}

lib/helpers/getConstructorName.js

@@ -12,4 +12,4 @@ module.exports = function getConstructorName(val) {
     return void 0;
   }
   return val.constructor.name;
-};
+};

lib/helpers/getDefaultBulkwriteResult.js

@@ -24,4 +24,4 @@ function getDefaultBulkwriteResult() {
   };
 }
 
-module.exports = getDefaultBulkwriteResult;
+module.exports = getDefaultBulkwriteResult;

lib/helpers/indexes/applySchemaCollation.js

@@ -10,4 +10,4 @@ module.exports = function applySchemaCollation(indexKeys, indexOptions, schemaOp
   if (schemaOptions.hasOwnProperty('collation') && !indexOptions.hasOwnProperty('collation')) {
     indexOptions.collation = schemaOptions.collation;
   }
-};
+};

lib/helpers/indexes/decorateDiscriminatorIndexOptions.js

@@ -11,4 +11,4 @@ module.exports = function decorateDiscriminatorIndexOptions(schema, indexOptions
     indexOptions.partialFilterExpression[discriminatorKey] = discriminatorName;
   }
   return indexOptions;
-};
+};

lib/helpers/indexes/isDefaultIdIndex.js

@@ -15,4 +15,4 @@ module.exports = function isDefaultIdIndex(index) {
 
   const key = get(index, 'key', {});
   return Object.keys(key).length === 1 && key.hasOwnProperty('_id');
-};
+};

lib/helpers/indexes/isTextIndex.js

@@ -13,4 +13,4 @@ module.exports = function isTextIndex(indexKeys) {
   }
 
   return isTextIndex;
-};
+};

lib/helpers/isMongooseObject.js

@@ -19,4 +19,4 @@ module.exports = function(v) {
       v.$isMongooseMap // Map
     )
   );
-};
+};

lib/helpers/isObject.js

@@ -13,4 +13,4 @@ module.exports = function(arg) {
     Buffer.isBuffer(arg) ||
     Object.prototype.toString.call(arg) === '[object Object]'
   );
-};
+};

lib/helpers/isPromise.js

@@ -3,4 +3,4 @@ function isPromise(val) {
   return !!val && (typeof val === 'object' || typeof val === 'function') && typeof val.then === 'function';
 }
 
-module.exports = isPromise;
+module.exports = isPromise;

lib/helpers/model/applyHooks.js

@@ -135,4 +135,4 @@ function applyHooks(model, schema, options) {
     objToDecorate[`$__${method}`] = middleware.
       createWrapper(method, originalMethod, null, customMethodOptions);
   }
-}
+}

lib/helpers/model/applyStaticHooks.js

@@ -68,4 +68,4 @@ module.exports = function applyStaticHooks(model, hooks, statics) {
       };
     }
   }
-};
+};

lib/helpers/once.js

@@ -9,4 +9,4 @@ module.exports = function once(fn) {
     called = true;
     return fn.apply(null, arguments);
   };
-};
+};

lib/helpers/path/flattenObjectWithDottedPaths.js

@@ -35,4 +35,4 @@ module.exports = function flattenObjectWithDottedPaths(obj) {
 
     flattenObjectWithDottedPaths(obj[key]);
   }
-};
+};

lib/helpers/path/parentPaths.js

@@ -15,4 +15,4 @@ module.exports = function parentPaths(path) {
   }
 
   return ret;
-};
+};

lib/helpers/path/setDottedPath.js

@@ -30,4 +30,4 @@ module.exports = function setDottedPath(obj, path, val) {
   if (!specialProperties.has(last)) {
     cur[last] = val;
   }
-};
+};

lib/helpers/pluralize.js

@@ -91,4 +91,4 @@ function pluralize(str) {
     }
   }
   return str;
-}
+}

lib/helpers/populate/SkipPopulateValue.js

@@ -7,4 +7,4 @@ module.exports = function SkipPopulateValue(val) {
 
   this.val = val;
   return this;
-};
+};

lib/helpers/populate/assignRawDocsToIdStructure.js

@@ -106,4 +106,4 @@ function assignRawDocsToIdStructure(rawIds, resultDocs, resultOrder, options, re
       rawIds[i] = doc;
     });
   }
-}
+}

lib/helpers/populate/assignVals.js

@@ -320,4 +320,4 @@ function isPopulatedObject(obj) {
 
 function noop(v) {
   return v;
-}
+}

lib/helpers/populate/createPopulateQueryFilter.js

@@ -77,4 +77,4 @@ function _formatMatch(match) {
     return Object.assign({}, match[0]);
   }
   return Object.assign({}, match);
-}
+}

lib/helpers/populate/getModelsMapForPopulate.js

@@ -726,4 +726,4 @@ function _findRefPathForDiscriminators(doc, modelSchema, data, options, normaliz
   }
 
   return modelNames;
-}
+}

lib/helpers/populate/leanPopulateMap.js

@@ -4,4 +4,4 @@
  * ignore
  */
 
-module.exports = new WeakMap();
+module.exports = new WeakMap();

lib/helpers/populate/lookupLocalFields.js

@@ -37,4 +37,4 @@ module.exports = function lookupLocalFields(cur, path, val) {
   }
 
   return cur[path];
-};
+};

lib/helpers/populate/markArraySubdocsPopulated.js

@@ -39,4 +39,4 @@ module.exports = function markArraySubdocsPopulated(doc, populated) {
       }
     }
   }
-};
+};

lib/helpers/populate/modelNamesFromRefPath.js

@@ -65,4 +65,4 @@ module.exports = function modelNamesFromRefPath(refPath, doc, populatedPath, mod
   modelNames = utils.array.flatten(modelNames);
 
   return modelNames;
-};
+};

lib/helpers/populate/removeDeselectedForeignField.js

@@ -28,4 +28,4 @@ module.exports = function removeDeselectedForeignField(foreignFields, options, d
       }
     }
   }
-};
+};

lib/helpers/populate/validateRef.js

@@ -16,4 +16,4 @@ function validateRef(ref, path) {
 
   throw new MongooseError('Invalid ref at path "' + path + '". Got ' +
     util.inspect(ref, { depth: 0 }));
-}
+}

lib/helpers/printJestWarning.js

@@ -14,4 +14,4 @@ if (typeof jest !== 'undefined' && process.nextTick.toString().indexOf('nextTick
     'with Jest\'s mock timers enabled. Please make sure you read ' +
     'Mongoose\'s docs on configuring Jest to test Node.js apps: ' +
     'https://mongoosejs.com/docs/jest.html');
-}
+}

lib/helpers/projection/applyProjection.js

@@ -74,4 +74,4 @@ function applyInclusiveProjection(doc, projection, hasIncludedChildren, projecti
     }
   }
   return ret;
-}
+}

lib/helpers/projection/hasIncludedChildren.js

@@ -33,4 +33,4 @@ module.exports = function hasIncludedChildren(fields) {
   }
 
   return hasIncludedChildren;
-};
+};

lib/helpers/projection/parseProjection.js

@@ -30,4 +30,4 @@ module.exports = function parseProjection(v, retainMinusPaths) {
   }
 
   return ret;
-};
+};

lib/helpers/query/applyGlobalOption.js

@@ -26,4 +26,4 @@ function applyGlobalOption(options, model, optionName) {
   } else if (utils.hasUserDefinedProperty(model.base.options, optionName)) {
     options[optionName] = model.base.options[optionName];
   }
-}
+}

lib/helpers/query/applyQueryMiddleware.js

@@ -75,4 +75,4 @@ function _getContexts(hook) {
     ret.document = hook.document;
   }
   return ret;
-}
+}

lib/helpers/query/cast$expr.js

@@ -279,4 +279,4 @@ function isLiteral(val) {
     return val.$literal != null;
   }
   return true;
-}
+}

lib/helpers/query/castFilterPath.js

@@ -51,4 +51,4 @@ module.exports = function castFilterPath(query, schematype, val) {
   }
 
   return val;
-};
+};

lib/helpers/query/sanitizeFilter.js

@@ -35,4 +35,4 @@ module.exports = function sanitizeFilter(filter) {
   }
 
   return filter;
-};
+};

lib/helpers/query/sanitizeProjection.js

@@ -11,4 +11,4 @@ module.exports = function sanitizeProjection(projection) {
       projection[keys[i]] = 1;
     }
   }
-};
+};