The purpose of this guide is to familiarize yourself with the oval development process. My hopes are that this guide will make contributing to the oval much simpler.
All of the information below assumes you've installed Gulp globally via npm install -g gulp-cli
. If you do
not want to install Gulp globally, just replace gulp
with node node_modules/gulp/bin/gulp.js
in the examples below.
Before contributing to oval, it makes sense to understand the development process. Git of course is a given so no time will be spent talking about it. oval uses Gulp as its task runner, which is used to build, lint, test, etc. Below are the gulp tasks:
lint
: Lint checks the necessary sources using ESLinttest
: Runs the tests
If you just run gulp
, all of these tasks mentioned above will be ran in the proper order. When working on oval
myself, I typically just run gulp test
while working on the bug fix or feature. Once I get the code ready to commit,
I will then run gulp
to lint check my code, run tests, ...
To submit new a new bug report, please follow these steps:
- Search that the bug hasn't already been reported
- File the bug (if the bug report is new)
Your bug report should meet the following criteria:
- Include a reproduction recipe (Document the steps required to reproduce the bug including any example code, etc.)
- Include what happens when the bug occurs
- Include what you expect to happen when the bug is fixed
In the end, please provide as much pertinent information as possible when describing the problem. A good bug report is clear, concise and requires no guess work on our part. Help us help you! (I couldn't resist...)
To submit a new PR, please follow these steps:
- Write a test to reproduce your bug or to test your enhancement/feature
- Write your code (I typically only run
gulp test
while working on the code until I get it done) - Run
gulp
- Commit
Your PR should meet the following criteria:
- Should include all generated sources
- Should pass lint checking and have all tests passing (We do have Travis CI setup to catch failing lint checks and failing tests but this is a safety net only)
- Should ideally be squashed into one commit (Regardless of how many commits, just make sure the commit messages are clear)
- Should include tests (Bug fixes and features should have tests included with them at all times)