Apply suggestions from code review

Co-authored-by: Tim Golen <tgolen@gmail.com>

lib/fastMerge.js

@@ -16,8 +16,6 @@ function isMergeableObject(val) {
 }
 
 /**
- * Merge Object and Arrays
- *
  * @param {{}|[]} target
  * @param {{}|[]} source
  * @returns {{}|[]}
@@ -32,17 +30,16 @@ function fastMerge(target, source) {
 }
 
 /**
-* Merge Object
-*
-* @param {{}} target
-* @param {{}} source
-* @returns {{}}
+ * @param {Object} target
+ * @param {Object} source
+ * @returns {Object}
 */
 mergeObject = function (target, source) {
  const destination = {};
  if (isMergeableObject(target)) {
+ // lodash adds a small overhead so we don't use it here
  // eslint-disable-next-line rulesdir/prefer-underscore-method
- const targetKeys = Object.keys(target); // lodash adds a small overhead so we don't use it here
+ const targetKeys = Object.keys(target);
  for (let i = 0; i < targetKeys.length; ++i) {
  const key = targetKeys[i];
  destination[key] = target[key];

lib/storage/providers/LocalForage.js

@@ -24,8 +24,9 @@ const provider = {
  return localforage.getItem(key)
  .then((existingValue) => {
  const newValue = _.isObject(existingValue)
+ // lodash adds a small overhead so we don't use it here
  // eslint-disable-next-line prefer-object-spread, rulesdir/prefer-underscore-method
- ? Object.assign({}, fastMerge(existingValue, value)) // lodash adds a small overhead so we don't use it here
+ ? Object.assign({}, fastMerge(existingValue, value))
  : value;
  return localforage.setItem(key, newValue);
  });