Thanks for being willing to contribute!
-
Do not open up a GitHub issue if the bug is a security vulnerability in Zap, and instead to refer to our Security Policy.
-
Ensure the bug was not already reported by searching on GitHub under Issues.
-
If you're unable to find an open issue addressing the problem, open a new one. Be sure to include a title and clear description, and as much relevant information as possible.
-
Open a new GitHub pull request with the patch.
-
Ensure the PR description clearly describes the problem and solution. Include the relevant issue number if applicable.
-
Before submitting, please read the Coding Guidelines to know more about our coding conventions and practices.
- Please join us on slack and check open issues to see what contributions are needed before tackling a task to avoid duplicate work.
- Ask any question about the Zap source code in slack.
We use two primary branches to manage development:
master
: Stable branch that corresponds to the current production release. Only bug fixes and release ready code will be merged into the master branch with a view to including in the next patch release (patch number increment).next
: Primary development branch. All new features should be developed against the next branch. The next branch will generally be quite far ahead of the master branch and includes all new features that are being developed for the next major release (major or minor number increment).
In general, most of the time you will want to develop against the next
branch.
We follow the conventional changelog standard for commit messages. You don't have to follow this convention if you don't like to. Just know that when we merge your commit, we'll probably use "Squash and Merge" so we can change the commit message :)
Valid conventional commit types are:
build
: Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)chore
: Other changes that don't modify src or test filesci
: Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)docs
: Documentation only changesfeat
: A new featurefix
: A bug fixperf
: A code change that improves performancerefactor
: A code change that neither fixes a bug nor adds a featurerevert
: Reverts a previous commitstyle
: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)test
: Adding missing tests or correcting existing tests
Example:
git commit -m "feat(close-channel): wire up close channel to UI"
Please make sure to run the tests before you commit your changes. You can run npm test
which will run the test suite.
There are git hooks set up with this project that are automatically installed when you install dependencies. They're really handy, with them in place you will get:
- Automatic reformatting of changed files on commit via prettier
- Automatic lint of changed files on commit (will not allow commit with files that fail lint) via eslint
- Automatic validation of commit message format (will not allow commit with invalid commit message) via commitlint
Branch names should start with a valid conventional commit type followed by /description_of_branch
.
Example:
git branch feat/close-channel-ui
When reviewing Pull Requests (PRs) you should use the following conventions to denote the level of review that you have performed:
Concept ACK
: Agree with the idea and overall direction, but haven't reviewed the code changes or tested them.utACK
: (untested ACK) Reviewed and agree with the code changes but haven't actually tested them.Tested ACK
: Reviewed the code changes and have verified the functionality or bug fix.ACK
: A loose ACK can be confusing. It's best to avoid them unless it's a documentation/comment only change in which case there is nothing to test/verify; therefore the tested/untested distinction is not there.NACK
: Disagree with the code changes/concept. Should be accompanied by an explanation.
This project has eslint rules and pull requests should pass npm run lint
before being merged. The eslint rules are not final by any means and can be changed if necessary.
Tests should try to be written for every feature/fix and pass npm run test
before being merged. With the demand for the Lightning Network and Zap rising, rapid development will naturally leave some code untested but we should all try our best.
Before making a new release, the version number should be incremented in the following files:
package.json
: The main repo package file.
If the release is a major or minor number increment (eg, 0.3.x
-> 0.4.x
), you should also update STABLE_VERSION
in the main app config file to reference the correct new stable version (this is used to determine the app's database namespace):
config/default.js
: The app config file.