Skip to content

Latest commit

 

History

History
32 lines (19 loc) · 1.24 KB

CONTRIBUTING.md

File metadata and controls

32 lines (19 loc) · 1.24 KB
Raw

Contributing to Origin

Thanks for helping out! 👍

Before you submit a PR, you'll want to make sure that:

  1. Any changes are tested.
  2. All tests pass. Run npm run test from the root of the repository to run all packages tests or run lerna run test --scope <package_name> to run the tests for a single package.
  3. The formatting is correct. Run npm run lint from the root of the repository.

If this is a new feature, make sure you've discussed it with our #engineering channel on Discord.

Coding style

Javascript

We use NPM style, as automated by the prettier tool. 2 space indents, no semi-semicolons.

Solidity

We use two space indents. Just copy the surrounding style and use your good judgement.

Coding guidelines

ETH addresses

Both checksummed and lowercased ETH address are technically valid. In our codebase, we chose the convention to:

  • Display ETH addresses in their checksummed format to the user.
  • Store ETH addresses in their lowercased format. That includes local storage and back-end storage such as Database, ElasticSearch, etc...
  • Use lowercased format when doing addresses comparisons.