- Development Setup
- Running Tests
- Coding Rules
- Commit Message Guidelines
- Writing Documentation
- Developing kepler.gl Website
This document describes how to set up your development environment to build and test Kepler.gl, and
explains the basic mechanics of using git
, node
, yarn
.
Before you can build Kepler.gl, you must install and configure the following dependencies on your machine:
-
Git: The Github Guide to Installing Git is a good source of information.
-
Node.js ^6.x: We use Node to generate the documentation, run a development web server, run tests, and generate distributable files. Depending on your system, you can install Node either from source or as a pre-packaged bundle.
We recommend using nvm (or nvm-windows) to manage and install Node.js, which makes it easy to change the version of Node.js per project.
-
Yarn: We use Yarn to install our Node.js module dependencies (rather than using npm). See the detailed installation instructions.
If you plan to contribute code to kepler.gl, you must have a GitHub account so you can push code and open Pull Requests in the GitHub Repository. You must fork the main kepler.gl repository to create a Pull Request.
To develop features, debug code, run tests, we use webpack to start a local web server and serve the kepler.gl demo app from the src directory.
# Clone your kepler.gl fork repository:
git clone git@github.com:<github username>/kepler.gl.git
# Go to the kepler.gl directory:
cd kepler.gl
# Add the main kepler.gl repository as an upstream remote to your repository:
git remote add upstream "git@github.com:keplergl/kepler.gl.git"
# Install JavaScript dependencies:
yarn --ignore-engines
# Setup mapbox access token locally
export MapboxAccessToken=<insert_your_token>
# Start the kepler.gl demo app
npm start
An demo app will be served at
http://localhost:8080/
This is the demo app we hosted on http://kepler.gl/#/demo. By default, it serves non-minified source code inside the src directory.
When develop, upgrade, debug deck.gl, Demo app can load deck.gl directly from src
// load deck.gl from node_modules/deck.gl/src, sub-modules from node_modules/@deck.gl/<module>/src
npm run start:deck
// load deck.gl src from the deck.gl folder parallel to kepler.gl
npm run start:deck-src
We write unit and browser tests with Tape and Enzyme, and lint with ESLint. Make sure to run test before submitting your PR. To run all of the tests once with node:
yarn test
To run separately
# lint
yarn lint
# node tests
yarn test-node
# browser tests
yarn test-browser
# fast test (no linting)
yarn fast-test
To generate a coverage report
yarn cover
To ensure consistency throughout the source code, keep these rules in mind as you are working:
- All features or bug fixes must be tested by one or more [specs][unit-testing].
- All public API methods must be documented with using jsdoc. To see how we document our APIs, please check out the existing source code and see the section about writing documentation
This project use Eslint together Prettier. The linter should automatically inform you if you break any rules (like incorrect indenting, line breaking or if you forget a semicolon). Before doing a pull request, make sure to run the linter.
# To run the linter
yarn lint
To commit your changes, please follow our rules over how our git commit messages can be formatted. This leads to more readable and unified messages that are easy to follow. But also, we use the git commit messages to generate the kepler.gl change log.
Each commit message consists of a header and a body. The header has a special format that includes a type and a subject. The PR # will be auto-generated once the PR is merged.
[<type>]<subject>(<pr>)
<BLANK LINE>
<body>
#e.g.
[Enhancement] Upgrade type-analyzer to pass 0/1 as integer (#317)
* Upgrade to type-analyzer@0.2.1
* Add test
The header is mandatory and the scope of the header is optional.
Any line of the commit message cannot be longer 100 characters! This allows the message to be easier to read on GitHub as well as in various git tools.
If the commit reverts a previous commit, it should begin with revert:
, followed by the header
of the reverted commit.
In the body it should say: This reverts commit <hash>.
, where the hash is the SHA of the commit
being reverted.
A commit with this format is automatically created by the git revert
command.
Must be one of the following, capitalized.
- [Feat]: A new feature
- [Enhancement]: An update of a existing feature
- [Bug]: A bug fix
- [Docs]: Documentation only changes
- [Style]: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, typos, etc)
- [Refactor]: A code change that neither fixes a bug nor adds a feature
- [Perf]: A code change that improves performance
- [Test]: Adding missing or correcting existing tests
- [Chore]: Changes to the build process or auxiliary tools and libraries such as documentation generation
The subject contains succinct description of the change:
- use the imperative, present tense: "change" not "changed" nor "changes"
- don't capitalize first letter
- no dot (.) at the end
Just as in the subject, use the imperative, present tense: "change" not "changed" nor "changes". The body should include the motivation for the change and contrast this with previous behavior.
Breaking Changes should start with the word BREAKING CHANGE:
with a space or two newlines.
The rest of the commit message is then used for this.
The Kepler.gl project uses jsdoc
This means that all the docs are stored inline in the source code and so are kept in sync as it changes.
There is also extra content (the developer guide, error pages, the tutorial, and misceallenous pages) that live inside the Kepler.gl repository as markdown files.
This means that since we generate the documentation from the source code, we can easily provide version-specific documentation by simply checking out a version of Kepler.gl and running the build.
We build Api docs from scratch using documentation.js. It generates docs from jsdoc:
yarn docs
You can find JSDoc instructions here. Documentation.js is interested in the following block tags:
-
@param {type} name description
- describes a parameter of a function -
@returns {type} description
- describes what a function returns -
@property
- describes a property of an object -
@description
- used to provide a description of a component in markdown -
@example
- specifies an example. -
@public
- Only methods with @public tag will be included in the docs
The type
in @param
and @returns
must be wrapped in {}
curly braces, e.g. {Object|Array}
.
Parameters can be made optional by either appending a =
to the type, e.g. {Object=}
, or by
putting the [name]
in square brackets.
Default values are only possible with the second syntax by appending =<value>
to the parameter
name, e.g. @param {boolean} [ownPropsOnly=false]
.
Make sure to export mapbox token in the same terminal before start the server.
export MapboxAccessToken=<insert_your_token>
In order to start
yarn web
To checkout the build
cd website && yarn build
Publish on github pages Authorized User Only.
important* Before publish. Copy the mapbox token at this link. (Only accessible by Uber developer). Deploy will fail if token is missing
export MapboxAccessToken=<insert_your_token>
yarn deploy
We currently host the demo-app on Github pages. We have provided a way to test github pages before pushing the branch to the actual repo. In order to test github pages with your changes, you need to satisfy the following requirements first:
- Make sure you have your own github pages (username.github.io) repo, click here
- In your local fork of kepler.gl, add your github pages repo to the list of git remotes by doing:
git remote add test git@github.com:<username>/<username>.github.io.git
With the above command, A new origin test will be created, and your own testing copy of gh pages will be push to it.
When everything is set up, run the following command:
yarn deploy:test
The above command will build the website and push to your gh-pages branch.