-
Notifications
You must be signed in to change notification settings - Fork 147
Contributing
Antony Budianto edited this page Jul 18, 2016
·
11 revisions
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
- 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
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 sharing it to your friends, giving stars, or pull requests! 😄