egjs is opened to everyone and we're welcoming for any kind of contribution. We believe that our project can grow with your interests helping others' necessities.
egjs has several style guidelines to follow. Before your start, please read attentively below instructions.
We adopted JSHint to maintain our code quality. All rules are described at .jshintrc file.
To maintain same code style across entire project, we adopted JSCS as code style checker. All conventions are described at .jscsrc file.
egjs use commit logs in many different purposes (like creating CHANGELOG, ease history searching, etc.). To not break, you'll be forced to follow our commit log guidelines. Before your commit/push, make sure following our commit log guidelines.
The outline is as below:
<type>(<module>): <subject>
<BLANK LINE>
<body>
<BLANK LINE>
<footer>
- Types
- feat: A new feature
- fix: A bug fix
- docs: Documentation only changes
- style: Changes that do not affect the meaning of the code. Such as white-space, formatting, missing semi-colons, etc... It also possible to change JSHint, JSCS rules of the code.
- refactor: A code change that neither fixes a bug nor adds a feature
- test: Adding missing tests. Changing tests.
- chore: Changes to the build process or tools and libraries such as documentation generation
See More Commit Log Guidelines
Steps to submit your pull request:
- Fork
egjs
on your repository - Create new branch from your egjs master branch (and be sure always to be up-to-date)
- Do your work
- Create test code for your work (when is possible)
- Run
grunt jshint
for linting. (update until without any error or warnings) - Run
grunt jscs
to check code style. (update until without any error or warnings) - Run test code by
grunt test OR grunt test:MODULE_NAME
. Make sure tests are all passed at least in Chrome(latest desktop version) and grunt qunit module. - Write commit log following convention and push to your repository branch.
- Create a new PR from your branch to egjs.
- Wait for reviews. When your contribution is well enough to be accepted, then will be merged to our branch.
- All done!
By contributing to egjs, you're agreeing that your contributions will be licensed under its MIT license.