2.9 KiB
Submitting Issues
If you are submitting a bug, please create a jsfiddle demonstrating the issue.
Read before submitting Pull Requests
- Pull requests to the
masterbranch will be closed. Please submit all pull requests to thedevelopbranch. - 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.
Code organization
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/**/*.jsmoment core filessrc/locale/*.jslocale filessrc/test/moment/*.jsmoment core testssrc/test/locale/*.jslocale 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.
Setting up development environment
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
Changing locale files
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).
Grunt tasks
We use Grunt for managing the build. Here are some useful Grunt tasks:
gruntThe default task lints the code and runs the tests. You should make sure you do this before submitting a PR.grunt testrun the tests.grunt releaseBuild everything, including minified files (do not include those in Pull Requests)grunt transpile:fr,ruBuild custom locale bundlesmoment-with-locales.custom.jsandlocales.custom.jsinsidebuild/umd/mincontaining just French and Russian.grunt sizePrint size statistics.