If you are submitting a bug, please create a jsfiddle demonstrating the issue.
- Pull requests to the
master
branch will be closed. Please submit all pull requests to thedevelop
branch. - Locale translations will not be merged without unit tests. See the British English unit tests for an example.
- Do not include the minified files in your pull request. These are
moment.js
,locale/*.js
,min/*.js
. Don't worry, we'll build them when we cut a release.
Starting from version 2.10.0 the code is placed under src/
.
moment.js
, locale/*.js
, min/*.js
are generated only on release.
DO NOT submit changes to the generated files. Instead only change
src/**/*.js
and run the tests.
src/lib/**/*.js
moment core filessrc/locale/*.js
locale filessrc/test/moment/*.js
moment core testssrc/test/locale/*.js
locale tests
We're using ES6 module system, but nothing else ES6, because of performance considerations (added code by the transpiler, less than optimal translation to ES5). So please do not use that fancy new ES6 feature in your patch, it won't be accepted.
To contribute, fork the library and install grunt and dependencies. You need git and node; you might use nvm or nenv to install node.
git clone https://github.com/moment/moment.git
cd moment
npm install -g grunt-cli
npm install
git checkout develop # all patches against develop branch, please!
grunt # this runs tests and jshint
If you have any changes to existing locale files, @mention
the original
author in the pull request (check the top of the language file), and ask if
he/she approves of your changes. Because I don't know any languages I can't
judge your locale changes, only the original author can :)
In order for your pull request to get merged it must have approval of original author, or at least one other native speaker has to approve of the change (happens rarely).
We use Grunt for managing the build. Here are some useful Grunt tasks:
grunt
The default task lints the code and runs the tests. You should make sure you do this before submitting a PR.grunt test
run the tests.grunt release
Build everything, including minified files (do not include those in Pull Requests)grunt transpile:fr,ru
Build custom locale bundlesmoment-with-locales.custom.js
andlocales.custom.js
insidebuild/umd/min
containing just French and Russian.grunt size
Print size statistics.