Skip to content

Latest commit

 

History

History
37 lines (23 loc) · 1.76 KB

CONTRIBUTING.md

File metadata and controls

37 lines (23 loc) · 1.76 KB
Raw

Contributing

Rules

There are a few basic ground-rules for contributors (including the maintainer(s) of the project):

  • No --force pushes or modifying the Git history in any way. If you need to rebase, ensure you do it in your own repo.
  • All modifications must be made in a pull-request to solicit feedback from other contributors.
  • Contributors should adhere to the Substrate coding style.

Reviewing pull requests

When reviewing a pull request, the end-goal is to suggest useful changes to the author. Reviews should finish with approval unless there are issues that would result in:

  • Buggy behavior.
  • Undue maintenance burden.
  • Breaking with house coding style.
  • Pessimization (i.e. reduction of speed as measured in the projects benchmarks).
  • Feature reduction (i.e. it removes some aspect of functionality that a significant minority of users rely on).
  • Uselessness (i.e. it does not strictly add a feature or fix a known issue).

Reviews may not be used as an effective veto for a PR because

  • There exists a somewhat cleaner/better/faster way of accomplishing the same feature/fix.
  • It does not fit well with some other contributors' longer-term vision for the project.

Releases

Declaring formal releases remains the prerogative of the project maintainer(s).

Changes to this arrangement

This is an experiment and feedback is welcome! This document may also be subject to pull-requests or changes by contributors where you believe you have something valuable to add or change.

Heritage

These contributing guidelines are modified from the "OPEN Open Source Project" guidelines for the Level project: https://github.com/Level/community/blob/master/CONTRIBUTING.md