X-Git-Url: http://git.bitcoin.ninja/index.cgi?a=blobdiff_plain;ds=sidebyside;f=CONTRIBUTING.md;h=9ddd792b238c57dae399b19d0d1a7abf86e12181;hb=6f5d81631ab9eb4967eebe4d2a8912903d40b541;hp=e8a57d85f7f7e8a9cebe857f6011e7c84dee6b6d;hpb=3aa0253f8ac58b6328fdecda74b78536046cbb65;p=rust-lightning diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index e8a57d85..9ddd792b 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -30,9 +30,13 @@ Getting Started First and foremost, start small. This doesn't mean don't be ambitious with the breadth and depth of your contributions but rather -understand the project context and culture before investing an asymmetric number of hours on +understand the project culture before investing an asymmetric number of hours on development compared to your merged work. +Browsing through the [meeting minutes](https://github.com/rust-bitcoin/rust-lightning/wiki/Meetings) +is a good first step. You will learn who is working on what, how releases are drafted, what are the +pending tasks to deliver, where you can contribute review bandwidth, etc. + Even if you have an extensive open source background or sound software engineering skills, consider that the reviewers' comprehension of the code is as much important as technical correctness. @@ -43,6 +47,8 @@ a "soft" commitment. If you're eager to increase the velocity of the dev process, reviewing other contributors work is the best you can do while waiting review on yours. +Also, getting familiar with the [glossary](GLOSSARY.md) will streamline discussions with regular contributors. + Contribution Workflow --------------------- @@ -69,8 +75,7 @@ be covered by functional tests. When refactoring, structure your PR to make it easy to review and don't hestitate to split it into multiple small, focused PRs. -The Minimal Supported Rust Version is 1.30.0 (enforced by our Travis and -GitHub Actions). +The Minimal Supported Rust Version is 1.36.0 (enforced by our GitHub Actions). Commits should cover both the issue fixed and the solution's rationale. These [guidelines](https://chris.beams.io/posts/git-commit/) should be kept in mind.