feature request: iterate over x, not just 1

[jduhls]

I made an adjustment to the iterate feature allowing programmer to control integer in the moment "add" method. Added a "step" argument. Not sure if best way to implement. Starting around line 198 in moment-range.js:

/**
 * Iterate over the date range by a given date range, executing a function
 * for each sub-range.
 *
 * @param {(!DateRange|String)} range Date range to be used for iteration or
 *                                    shorthand string (shorthands:
 *                                    http://momentjs.com/docs/#/manipulating/add/)
 * @param {!DateRange~by} hollaback Callback
 * @param {!boolean} exclusive Indicate that the end of the range should not
 *                             be included in the iter.
 * @param {!Integer} step Indicate number of moment shorthand strings to moment.add
 *
 * @return {DateRange} `this`
 */
DateRange.prototype.by = function(range, hollaback, exclusive, step) {
  if (typeof range === 'string') {
    _byString.call(this, range, hollaback, exclusive, step);
  }
  else {
    _byRange.call(this, range, hollaback, exclusive);
  }
  return this;
};


/**
 * Callback executed for each sub-range.
 *
 * @callback DateRange~by
 *
 * @param {!Moment} current Current moment object for iteration
 */

/**
 * @private
 */
function _byString(interval, hollaback, exclusive, step) {
  var current = moment(this.start);

  while (this.contains(current, exclusive)) {
    hollaback.call(this, current.clone());
    if((typeof step === 'number') && Math.floor(step) === step){
      current.add(step, interval);
    } else {
      current.add(1, interval);
    }

  }
}

...p.s. I like "hollaback".

[gf3]

awesome suggestion—i've added this functionality to the upcoming v3 release (see the commit above)

[gf3]

just opened a PR for 3.0.0 which incorporates these changes. feel free to leave feedback or raise any questions you may have. thanks again for your contributions