From mboxrd@z Thu Jan 1 00:00:00 1970 From: Richard Ipsum Subject: [PATCH 2/2] contrib/git-candidate: Add README Date: Tue, 10 Nov 2015 12:56:38 +0000 Message-ID: <1447160198-23296-3-git-send-email-richard.ipsum@codethink.co.uk> References: <1447160198-23296-1-git-send-email-richard.ipsum@codethink.co.uk> Cc: Richard Ipsum To: git@vger.kernel.org X-From: git-owner@vger.kernel.org Tue Nov 10 13:57:00 2015 Return-path: Envelope-to: gcvg-git-2@plane.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1Zw8Tr-0000KT-N6 for gcvg-git-2@plane.gmane.org; Tue, 10 Nov 2015 13:57:00 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752331AbbKJM44 (ORCPT ); Tue, 10 Nov 2015 07:56:56 -0500 Received: from ducie-dc1.codethink.co.uk ([185.25.241.215]:45318 "EHLO ducie-dc1.codethink.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751284AbbKJM44 (ORCPT ); Tue, 10 Nov 2015 07:56:56 -0500 Received: from localhost (localhost [127.0.0.1]) by ducie-dc1.codethink.co.uk (Postfix) with ESMTP id F0142461947; Tue, 10 Nov 2015 12:56:54 +0000 (GMT) X-Virus-Scanned: Debian amavisd-new at ducie-dc1.codethink.co.uk Received: from ducie-dc1.codethink.co.uk ([127.0.0.1]) by localhost (ducie-dc1.codethink.co.uk [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id w3sLVT5mkfwu; Tue, 10 Nov 2015 12:56:52 +0000 (GMT) Received: from salo.dyn.ducie.codethink.co.uk. (salo.dyn.ducie.codethink.co.uk [10.24.1.218]) by ducie-dc1.codethink.co.uk (Postfix) with ESMTPSA id 816084620D7; Tue, 10 Nov 2015 12:56:50 +0000 (GMT) X-Mailer: git-send-email 2.1.4 In-Reply-To: <1447160198-23296-1-git-send-email-richard.ipsum@codethink.co.uk> Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: Describes motivation for git-candidate and shows an example workflow. Signed-off-by: Richard Ipsum --- contrib/git-candidate/README.md | 154 ++++++++++++++++++++++++++++++++++++++++ 1 file changed, 154 insertions(+) create mode 100644 contrib/git-candidate/README.md diff --git a/contrib/git-candidate/README.md b/contrib/git-candidate/README.md new file mode 100644 index 0000000..d2d4437 --- /dev/null +++ b/contrib/git-candidate/README.md @@ -0,0 +1,154 @@ +git-candidate +============= + +git-candidate provides candidate review and patch tracking, +it differs from other tools that provide this by storing _all_ +content within git. + +## Why? + +Existing tools such as Github's pull-requests and Gerrit are already +in wide use, why bother with something new? + +We are concerned that whilst git is a distributed version control +system the systems used to store comments and reviews for content +under version control are usually centralised, +git-candidate aims to solve this by storing +all patch-tracking data in git proper. + +## Example review process + +### Contributor - Submits a candidate + + (hack hack hack) + + (feature)$ git commit -m "Add archived repo" + (feature)$ git candidate create archivedrepo master + -m "Add support for archived repo" + Candidate archivedrepo created successfully. + (feature)$ git candidate submit origin archivedrepo + Candidate was submitted successfully. + +### Upstream - Reviews candidate + + (master)$ git candidate fetch origin + (master)$ git candidate status origin/archiverepo + Revision: 6239bd72d597357af901718becae91cee2a32b73 + Ref: candidates/origin/archiverepo + Status: active + Land: master + + Add archived repo support + + lib/gitano/command.lua | 28 ++++++++++++++++++++++------ + 1 file changed, 22 insertions(+), 6 deletions(-) + + (master)$ git show candidates/origin/archiverepo + commit 2db28539c8fa7b81122382bcc526c6706c9e113a + Author: Richard Ipsum + Date: Thu Oct 8 10:43:22 2015 +0100 + + Add support for archived repository masking in `ls` + + By setting `project.archived` to something truthy, a repository + is thusly masked from `ls` output unless --all is passed in. + + Signed-off-by: Richard Ipsum + .... + .... + + + (master)$ git candidate review origin/archiverepo --vote -1 + -m "Sorry, I'll need to see tests before I can accept this" + (master)$ git candidate submit origin archiverepo + Review added successfully + +### Contributor - Revises candidate + + (master)$ git candidate fetch origin + (master)$ git candidate status origin/archiverepo + Revision: 6239bd72d597357af901718becae91cee2a32b73 + Ref: candidates/origin/archiverepo + Status: active + Land: master + + Add archived repo support + + lib/gitano/command.lua | 28 ++++++++++++++++++++++------ + 1 file changed, 22 insertions(+), 6 deletions(-) + + -------------------------------------------------------------------------------- + 1 review + -------------------------------------------------------------------------------- + + Author: Emmet Hikory + Date: Tue Oct 13 10:09:45 2015 +0100 + Vote: -1 + + Sorry, I'll need to see tests before I can accept this + + -------------------------------------------------------------------------------- + + (hack hack hack add tests) + + (feature_v2)$ git log --oneline -1 + Ensure the `ls` yarn checks for archived repos + + (feature_v2)$ git candidate revise origin/archiverepo + -m "Add archived repo support with tests" + Candidate archiverepo revised successfully. + + (feature_v2)$ git candidate submit origin archiverepo + Candidate was submitted successfully. + +### Upstream - Merges candidate + + (master)$ git candidate fetch origin + (master)$ git candidate status origin/archiverepo + Revision: 4cd3d1197d399005a713ca55f126a9086356a072 + Ref: candidates/origin/archiverepo + Status: active + Land: master + + Add archived repo support with tests + + lib/gitano/command.lua | 28 ++++++++++++++++++++++------ + testing/02-commands-ls.yarn | 19 +++++++++++++++++++ + 2 files changed, 41 insertions(+), 6 deletions(-) + + (master)$ git candidate review origin/archiverepo --vote +2 + -m "Looks good, merging. Thanks for your efforts" + Review added successfully + + (master)$ git candidate submit origin archiverepo + Candidate was submitted successfully. + + (master)$ git merge candidates/origin/archiverepo + (master)$ git push origin master + +### Contributor - Observes candidate has been accepted + + (feature_v2)$ git candidate fetch origin + (feature_v2)$ git candidate status origin/archiverepo + Revision: 4cd3d1197d399005a713ca55f126a9086356a072 + Ref: candidates/origin/archiverepo + Status: active + Land: master + + Add archived repo support with tests + + lib/gitano/command.lua | 28 ++++++++++++++++++++++------ + testing/02-commands-ls.yarn | 19 +++++++++++++++++++ + 2 files changed, 41 insertions(+), 6 deletions(-) + + -------------------------------------------------------------------------------- + 1 review + -------------------------------------------------------------------------------- + + Author: Emmet Hikory + Date: Tue Oct 13 10:35:00 2015 +0100 + Vote: +2 + + Looks good, merging. Thanks for your efforts + + -------------------------------------------------------------------------------- -- 2.1.4