Add moment.js to peerDependencies

[jchavarri]

Really minor :) for consumers to get the original moment package when downloading the bindings and avoid version mismatches, it should be better moved to dependencies (now it's in devDependencies). Edit: better add it to peerDeps, see below 👇

[jimexist]

for consumers to get the original moment package when downloading

i think the users should try to add moment in their own package.json

avoid version mismatches

a better solution is to include moment in peer dependencies. but for dev dependencies it's for the sake of developing in this repo.

[jchavarri]

Interesting. I guess I was thinking on cases where you start a project from scratch, but peerDependencies definitely makes more sense for JavaScript projects that already exists, and are progressively migrated to Reason / OCaml, as there are instances of moment.js objects being passed around. This article helped me understand better when to use peerDependencies, depending if abstract types are being accepted or returned from the public interface contract.

It seems other projects in a similar situation like grunt-contrib-clean also have devDependencies and peerDependencies (relevant issue)

[jimexist]

@jchavarri FYI this is now added