guyellis / nth-day Goto Github PK

nthDay(3, 'friday', ...) or nthDay(3, 'fri', ...) in addition to nthDay(3, 5, ...)

...for cross-platform compatibility.

When I originally wrote this using moment was the easiest and quickest way to get this functional for what I wanted.

To give this a broader appeal and to make this a dependency-free module I think that it would be a good idea to remove the moment dependency.

Instead of using moment and returning a moment object a JS Date object should be returned.

Version 6.25.0 of babel-core just got published.

This version is covered by your current version range and after updating it in your project the build failed.

As babel-core is “only” a devDependency of this project it might not break production or downstream projects, but “only” your build or test tools – preventing new deploys or publishes.

I recommend you give this issue a high priority. I’m sure you can resolve this 💪

Your Greenkeeper Bot 🌴