PouchDB - The JavaScript Database that Syncs
Welcome, so you are thinking about contributing to PouchDB? awesome, this is a great place to start.
The following documentation should answer most of the common questions about how to get starting contributing, if you have any questions, please feel free to ask on the PouchDB Mailing List or in #pouchdb on irc.freenode.net.
Most project discussions should happen on the Mailing list / Bug Tracker and IRC, however if you are a first time contributor and want some help getting started feel free to send a private email to any of the following maintainers:
- Dale Harvey (dale@arandomurl.com, daleharvey on IRC)
- Calvin Metcalf (calvin.metcalf@gmail.com)
If you are looking for something to work on, we try to maintain a list of issues that should be suitable for first time contributions, they can be found tagged goodfirstpatch.
- Almost all Pull Requests for features or bug fixes will need tests
- We follow Felix's Node.js Style Guide
- Almost all Pull Requests for features or bug fixes will need tests (seriously, its really important)
- Before opening a pull request run
$ npm test
to lint test the changes and run node tests. Preferably run the browser tests as well. - Commit messages should follow the following style:
(#99) - A brief one line description < 50 chars
Followed by further explanation if needed, this should be wrapped at
around 72 characters. Most commits should reference an existing
issue
PouchDB needs the following to be able to build and test your build, if you haven't installed them then best to do do so now, we will wait.
All dependancies installed? great, now building PouchDB itself is a breeze:
$ cd pouchdb
$ npm install
$ npm run build
You will now have various distributions of PouchDB in your dist
folder, congratulations.
- If you are on windows, you will need
node-gyp
to install levelup, visit https://github.com/TooTallNate/node-gyp#installation for installation instructions.
The PouchDB test suite expects an instance of CouchDB running in Admin Party on http://127.0.0.1:5984, you can configure this by sending the COUCH_HOST
env var.
- PouchDB has been primarily developed on Linux and OSX, if you are using Windows then these instructions will have problems, we would love your help fixing them though.
Run all tests with:
$ npm test
Browser tests can be run automatically with:
$ CLIENT=firefox npm test
or you can run:
$ npm run dev
and open http://127.0.0.1:8000/tests/test.html in your browser of choice.
$ GREP=test.replication.js npm test
or append ?grep=test.replication.js
if you opened the tests in a browser manually
$ COVERAGE=1 npm test
$ COUCH_HOST=http://user:pass@myname.host.com npm run dev
or
$ COUCH_HOST=http://user:pass@myname.host.com npm test
For quick debugging, you can run an interactive Node shell with the PouchDB
variable already available:
npm run shell
PouchDB is looking to support alternative backends that comply with the LevelDOWN API. For example, simply include LEVEL_BACKEND=leveljs
in your npm run build
and npm run dev
commands to experiment with this feature!
Doing so will also create a separate distribution, for example, pouchdb-leveljs.js
rather than pouchdb-nightly.js
. In order to test a different distribution from pouchdb-nightly.js
, you must specify in the testing URL: http://127.0.0.1:8000/tests/test.html?sourceFile=pouchdb-leveljs.js. LEVEL_BACKEND=leveljs npm run test
will accomplish the same thing.
Workflows can vary, but here is a very simple workflow for contributing a bug fix:
$ git clone git@github.com:myfork/pouchdb.git
$ git remote add pouchdb https://github.com/daleharvey/pouchdb.git
$ git checkout -b 121-issue-keyword master
# Write tests + code
$ git add src/afile.js
$ git commit -m "(#121) - A brief description of what I changed"
$ git push origin 121-issue-keyword
The source for the website http://pouchdb.com is stored inside the docs
directory of the PouchDB repository, you can make changes and submit pull requests as with any other patch. To build and view the website locally you will need to install jekyll then:
$ npm run build-site
You should now find the documentation at http://127.0.0.1:4000
With great power comes great responsibility yada yada yada:
- Code is peer reviewed, you should (almost) never push your own code.
- Please don't accidentally force push to master.
- Cherry Pick / Rebase commits, don't use the big green button.
- Ensure reviewed code follows the above contribution guidelines, if it doest feel free to amend and make note.
- Please try to watch when Pull Requests are made and review and / or commit them in a timely manner.
- After you merge in a patch use tin to update the version accordingly. Run
tin -v x.x.x-prerelease
with x.x.x being the previous version upgraded appropriately via semver. When we are ready to publish to npm we can remove the-prerelease
. - Thanks, you are all awesome human beings.