ARRRRR, don't be a filthy landlubber, contribute to pirates!
ARRRRR, thanks for filing an issue matey. Make sure to include the version!
ARRRRRRRR, thanks so much for contributing to pirates, matey. Here's what you need to know:
- Pirates is in
lib/
, tests are intest
(see below for more on the tests) - Test and lint your code with
npm test
.
- We will not accept any code that does not have (and pass) tests for it, or fails linting (a few warnings are OK).
- Only known-good code will be merged into master, because...
- We use
semantic-release
, so whatever is in master is automatically published to npm. - To make semantic-release work, we use the Angular commit message guidelines. Please make sure that all commits that you make follow them.
- To assist with this, we support Commitizen, which makes committing easy.
- If you have a commit that doesn't comply with the guidelines,
walk the plank!No Problem!git commit --amend and
git rebase -i ` are your friends.
See test/README.md
for lots of good info on tests.