summaryrefslogtreecommitdiffstats
path: root/doc
diff options
context:
space:
mode:
authorStephen Worley <sworley@cumulusnetworks.com>2019-12-04 17:27:30 +0100
committerStephen Worley <sworley@cumulusnetworks.com>2019-12-04 17:27:30 +0100
commit118cf7edb020cb56588cc62052089faf0cc15969 (patch)
tree4082ee33b4f3f88edd36b09883b5b2c218152678 /doc
parentMerge pull request #5352 from donaldsharp/5013_cleanup_merge_issues (diff)
downloadfrr-118cf7edb020cb56588cc62052089faf0cc15969.tar.xz
frr-118cf7edb020cb56588cc62052089faf0cc15969.zip
doc: add doc for easy commit sign-off with `-s`
Add some doc to let developers know about the `-s` flag with `git commit`. We were seeing some people writing the sign-off manually. Signed-off-by: Stephen Worley <sworley@cumulusnetworks.com>
Diffstat (limited to 'doc')
-rw-r--r--doc/developer/workflow.rst5
1 files changed, 5 insertions, 0 deletions
diff --git a/doc/developer/workflow.rst b/doc/developer/workflow.rst
index 6eef7532b..40378f021 100644
--- a/doc/developer/workflow.rst
+++ b/doc/developer/workflow.rst
@@ -293,6 +293,11 @@ Code submitted to FRR must be signed off. We have the same requirements for
using the signed-off-by process as the Linux kernel. In short, you must include
a ``Signed-off-by`` tag in every patch.
+An easy way to do this is to use ``git commit -s`` where ``-s`` will automatically
+append a signed-off line to the end of your commit message. Also, if you commit
+and forgot to add the line you can use ``git commit --amend -s`` to add the
+signed-off line to the last commit.
+
``Signed-off-by`` is a developer's certification that they have the right to
submit the patch for inclusion into the project. It is an agreement to the
:ref:`Developer's Certificate of Origin <developers-certificate-of-origin>`.