Skip to content

Latest commit

 

History

History
111 lines (72 loc) · 3.5 KB

CONTRIBUTING.md

File metadata and controls

111 lines (72 loc) · 3.5 KB
Raw

Contributing to BitHodl

We're glad that you'd like to help us make BitHodl even better. But first, read these guidelines and instructions.

Setting up the environment

Install Node.js and npm

Install Node.js (v8) using nvm:

$ nvm install v8

Install Xcode

Download and install Xcode from developer.apple.com.

Clone the repository

$ git clone https://github.com/blockfirm/bithodl-app.git
$ cd bithodl-app

Install dependencies

$ npm install
$ npm install -g react-native-cli
$ react-native link

Coding conventions

Follow the coding conventions defined in .eslintrc. Run npm run lint to verify that your code follows the rules.

EditorConfig

Use EditorConfig so that your code automatically comply with our coding style.

Submitting pull requests

When submitting pull requests, please follow these rules:

  • Make sure there isn't another pull request that already does the same thing
  • Make sure you've followed our coding conventions
  • Provide tests for your code
  • Run the tests before submitting
  • Follow the commit message conventions
  • Rebase your commits if possible
  • Refer to a GitHub Issue
  • Include instructions on how to verify/test your changes
  • Include screenshots for GUI changes

Commit message conventions

  • Briefly describe what your changes does
  • Use the imperative, present tense: "change" not "changed" nor "changes"
  • Capitalize the first letter
  • No dot (.) at the end

Testing

We're using Jest to test all JavaScript code. Always run the tests before and after you start changing the code. This will ensure high quality of the code. Also write new tests when needed and be sure to follow our test conventions when writing them.

$ npm test

Testing conventions

  • Put all tests in the __tests__ folder with the same folder structure as in src/
  • Put all mocks in the __mocks__ folder
  • Give the file the same name as the file under test, but append .test, e.g. App.test.js
  • If you are unsure, read this post on how to write good unit tests

Code coverage

Aim for 100% code coverage when writing tests. Run the following command to display the current code coverage:

$ npm run test-coverage

Releases

We're using the Semantic Versioning Specification when releasing new versions.

Roadmap

We're using Issues on GitHub to plan our milestones. That include new features as well as bug fixes.

Reporting security issues

We appreciate all security related reports. Please send them to hello@bithodl.com.

Reporting bugs

To report a bug or another issue, please submit a new issue on GitHub.

Describe your issue with steps of how to reproduce it, and if not obvious, what the expected behavior should be. Also include information about your system (e.g. version and OS) and your setup.

Discussion and feedback

As there are no channels set up for general discussion, we are using GitHub Issues for now. Tag your issue as a Question. You can also tweet us at @bithodl.