Skip to content

Latest commit

 

History

History
81 lines (64 loc) · 3.08 KB

CONTRIBUTING.md

File metadata and controls

81 lines (64 loc) · 3.08 KB
Raw

How to contribute to billboard.js

billboard.js 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.

Style Guide

billboard.js has several style guidelines to follow. Before your start, please read attentively below instructions.

Linting and Code Convention

To maintain the code style, we adopted ESLint to maintain our code quality. The rules are modified version based on Airbnb JavaScript Style Guide.

Commit Log Guidelines

billboard.js use formatted commit logs for 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.
    • 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
    • skip: For commits made by after the 1st commit. Usually for applying code review changes.

  • Body

A short descriptive message which commit consists. At the end preferably(or should) contain related github issue number referencing by Ref #ISSUE-NO.

ex) When the commit is about issue number 20, then

Ref #20

Example commit of fixing a bug:

Fix(Axis): Correct tick rendering

Update condition of tick to avoide unnecessary tick rendering

Ref #20

Example commit applying code review (after the 1st commit)

Note: Commit log starting with skip: type will be ignored by commit hook constraint.

skip: Applied the review

How to submit Pull Requests

Steps to submit your pull request:

  1. Fork billboard.js on your repository
  2. Create new branch from your billboard.js master branch (and be sure always to be up-to-date)
  3. Do your work
  4. Create test code for your work (when is possible)
  5. Run npm run lint for linting and code style check. (update until without any error or warnings)
  6. Run test code by npm test or npm test:chrome to run the test on chrome browser. Make sure tests are all passed at least in latest version of Chrome(mobile/desktop).
  7. Write commit log following convention and push to your repository branch.
  8. Create a new PR from your branch to billboard.js.
  9. Wait for reviews. When your contribution is well enough to be accepted, then will be merged to our branch.
  10. All done!

License

By contributing to billboard.js, you're agreeing that your contributions will be licensed under its MIT license.