-
Notifications
You must be signed in to change notification settings - Fork 18
Contributing
Antony Budianto edited this page Sep 20, 2016
·
2 revisions
- We only process starter bugs and features, for problem/question please ask on StackOverflow
- We won't process issues if not following given template, unless it's a feature proposal
- You can see the guidelines in the issue message body when creating it
"A problem well stated is a problem well solved. - Charles F. Kettering"
It's very welcomed for you to contribute to this starter by sending a pull request, and to make sure the process run smoothly, I'd like you to follow these guidelines:
- We don't accept strong opinionated requests, though we will consider it for extension instead
- Please rebase your branch against the current master
- Make sure you have the latest dependencies installed by running
npm install
before - All tests must passed, and not lowering coverage percentage
- Link to the related issues to automatically close the issue when it's merged
- example: fix #123 in the pull request body message
- Use semantic commit message where possible
- Please file an issue first to discuss it. If it's accepted, then you can continue to work on the pull request
- Always include the unit test
This starter may not fit for your workflow, since it's opinionated. Therefore you can always fork and custom it to fit your workflow
⭐ Support this starter by giving stars, sharing to your friends, or contributing