grassrootseconomics/openethereum
5
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity
78916beb1a
openethereum/.github/CONTRIBUTING.md
Andronik Ordian 10d82ef119
initial cleanup (#11542) 
* cleanup

* upbork

* review grumble: rename client id prefix

* rpc-tests: make one of the clients open-ethereum

* fix doctest

* README: mention clang version and that our wiki is old
2020-03-09 15:07:15 +01:00

4.2 KiB
Raw Blame History

Contributing Guidelines

Do you have a question?

Check out our Basic Usage, Configuration, and FAQ articles on our wiki!

See also frequently asked questions tagged with parity on Stack Exchange.

Report bugs!

Do not open an issue on Github if you think your discovered bug could be a security-relevant vulnerability. Please, read our security policy instead.

Otherwise, just create a new issue in our repository and state:

  • What's your Open Ethereum version?
  • What's your operating system and version?
  • How did you install Open Ethereum?
  • Is your node fully synchronized?
  • Did you try turning it off and on again?

Also, try to include steps to reproduce the issue and expand on the actual versus expected behavior.

Contribute!

If you would like to contribute to Open Ethereum, please fork it, fix bugs or implement features, and propose a pull request.

Labels & Milestones

We use labels to manage PRs and issues and communicate the state of a PR. Please familiarize yourself with them. Furthermore we are organizing issues in milestones. Best way to get started is to a pick a ticket from the current milestone tagged easy and get going, or mentor and get in contact with the mentor offering their support on that larger task.

Rules

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

  • No pushing directly to the master branch.
  • All modifications must be made in a pull-request to solicit feedback from other contributors.
  • Pull-requests cannot be merged before CI runs green and two reviewers have given their approval.
  • Contributors should adhere to the Parity Ethereum Style Guide.

Recommendations

  • Non-master branch names should be prefixed with a short name moniker, followed by the associated Github Issue ID (if any), and a brief description of the task using the format <GITHUB_USERNAME>-<ISSUE_ID>-<BRIEF_DESCRIPTION> (e.g. gavin-123-readme). The name moniker helps people to inquiry about their unfinished work, and the GitHub Issue ID helps your future self and other developers (particularly those who are onboarding) find out about and understand the original scope of the task, and where it fits into Open Ethereum Projects.
  • Remove stale branches periodically

Preparing Pull Requests

  • If your PR does not alter any logic (e.g. comments, dependencies, docs), then it may be tagged insubstantial.

  • Once a PR is ready for review please add the pleasereview label.

Reviewing Pull Requests*:

  • At least two reviewers are required to review PRs (even for PRs tagged insubstantial).

When doing a review, make sure to look for any:

  • Buggy behavior.
  • Undue maintenance burden.
  • Breaking with house coding style.
  • Pessimization (i.e. reduction of speed as measured in the projects benchmarks).
  • Breaking changes should be carefuly reviewed and tagged as such so they end up in the changelog.
  • Uselessness (i.e. it does not strictly add a feature or fix a known issue).

License.

By contributing to Open Ethereum, you agree that your contributions will be licensed under the GPLv3 License.