summaryrefslogtreecommitdiff
path: root/docs/contributing.md
diff options
context:
space:
mode:
Diffstat (limited to 'docs/contributing.md')
-rw-r--r--docs/contributing.md27
1 files changed, 22 insertions, 5 deletions
diff --git a/docs/contributing.md b/docs/contributing.md
index 7bbc186d0..daabcf055 100644
--- a/docs/contributing.md
+++ b/docs/contributing.md
@@ -9,6 +9,9 @@
- If you are going to work on an issue, mention so in the issue comments
_before_ you start working on the issue.
+- If you are going to work on a new feature, create an issue and discuss with
+ other contributors _before_ you start working on the feature.
+
- Please be professional in the forums. We follow
[Rust's code of conduct](https://www.rust-lang.org/policies/code-of-conduct)
(CoC). Have a problem? Email ry@tinyclouds.org.
@@ -22,11 +25,25 @@ Instructions on how to build from source can be found
Before submitting, please make sure the following is done:
-1. That there is a related issue and it is referenced in the PR text.
-2. There are tests that cover the changes.
-3. Ensure `cargo test` passes.
-4. Format your code with `./tools/format.py`.
-5. Make sure `./tools/lint.py` passes.
+1. Give the PR a descriptive title.
+
+Examples of good PR title:
+
+- fix(std/http): Fix race condition in server
+- docs(console): Update docstrings
+- feat(doc): Handle nested re-exports
+
+Examples of bad PR title:
+
+- fix #7123
+- update docs
+- fix bugs
+
+2. Ensure there is a related issue and it is referenced in the PR text.
+3. Ensure there are tests that cover the changes.
+4. Ensure `cargo test` passes.
+5. Ensure `./tools/format.py` passes without changing files.
+6. Ensure `./tools/lint.py` passes.
## Changes to `third_party`