From e6bf20978335f253b3e9d19ff24d9035e65bd7cf Mon Sep 17 00:00:00 2001 From: Mike Voronov Date: Tue, 26 Apr 2022 16:22:42 +0300 Subject: [PATCH] Add developer notes re PR reviewing (#15) * add developer notes * Update docs/developer-notes.md Co-authored-by: Valery Antopol Co-authored-by: Valery Antopol --- docs/developer-notes.md | 5 +++++ 1 file changed, 5 insertions(+) create mode 100644 docs/developer-notes.md diff --git a/docs/developer-notes.md b/docs/developer-notes.md new file mode 100644 index 0000000..43bd106 --- /dev/null +++ b/docs/developer-notes.md @@ -0,0 +1,5 @@ +In marine-rs-sdk-test we use the following conventions for PR reviewing: +- **Concept ACK** - agree with the idea and overall concept, but haven't reviewed the code changes or tested them, +- **utACK (untested ACK)** - reviewed and agree with the code changes but haven't actually tested them, +- **Tested ACK** - reviewed the code changes and have verified the functionality or a bug fix, +- **NACK** - disagree with the code changes/concept, should be accompanied by an explanation.