Hi! We are really excited that you are interested in contributing to Apostille-library. Before submitting your contribution though, please make sure to take a moment and read through the following guidelines.
- Code of Conduct
- Issue Reporting Guidelines
- Pull Request Guidelines
- Development Setup
- Project Structure
- Please try to stick to the template when creating a new issues.
-
The
master
branch is basically just a snapshot of the latest stable release. All development should be done in dedicated branches. Do not submit PRs against themaster
branch. -
First checkout to the
develop
branch, and start a new branchfeature/YOUR_FEATUR
once done merge back againstdevelop
. -
Work in the
src
folder and DO NOT checkindist
in the commits. -
It's OK to have multiple small commits as you work on the PR - we will let GitHub automatically squash it before merging.
-
Make sure
npm test
passes. (see development setup) -
If adding new feature:
- Add accompanying test case.
- Provide convincing reason to add this feature. Ideally you should open a suggestion issue first and have it greenlighted before working on it.
-
If fixing a bug:
- If you are resolving a special issue, add
(fix #xxxx[,#xxx])
(#xxxx is the issue id) in your PR title for a better release log, e.g.update entities encoding/decoding (fix #3899)
. - Provide detailed description of the bug in the PR. Live demo preferred.
- Add appropriate test coverage if applicable.
- If you are resolving a special issue, add
You will need Node.js version 6+
After cloning the repo, run:
$ npm install # or yarn
Commit messages should follow the commit message convention so that changelogs can be automatically generated. Commit messages will be automatically validated upon commit. If you are not familiar with the commit message convention, you should use npm run commit
instead of git commit
, which provides an interactive CLI for generating proper commit messages.
# interactive CLI for generating proper commit messages
$ npm run commit
# run a lint check against the code
$ npm run lint
# build the projects and genrate a new dist folder
$ npm run build
# run the full test suite, with code coverage report
$ npm test
# run linting and testing
npm run validate
Please make sure to have this pass successfully before submitting a PR.
Thank you to all the people who have already contributed to Apostille-libray!