numam-dpdk/doc/guides/contributing
Harry van Haaren f9ef083c6c doc: add author on cc to git fixline alias
With this commit, the correct method to use git fixline to indicate
a fix of a previous commit has changed. The new rules require the
author of the original code that is being fixed to be on CC.

The logic behind this improvement is that if there is a genuine bug,
one of the ideal people to review is the author of the original code
being fixed. Adding them on Cc makes them aware of the patch, avoiding
it from being passed by accidentally while reading the mailing-list.

Given that the original author (now on Cc:) might not actually review,
there is no value in keeping the Cc: in git commit history. If the
original author performs a review, their Reviewed-by: or Acked-by: is
stored in git history (same as now).

Signed-off-by: Harry van Haaren <harry.van.haaren@intel.com>
Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
Acked-by: Thomas Monjalon <thomas@monjalon.net>
2017-08-03 15:27:26 +02:00
..
img doc: add patch submit cheatsheet 2015-12-14 23:17:33 +01:00
cheatsheet.rst doc: add patch submit cheatsheet 2015-12-14 23:17:33 +01:00
coding_style.rst eal: support dynamic log types 2017-04-05 13:37:17 +02:00
design.rst doc: update for VFD experimental API 2017-01-17 19:48:02 +01:00
documentation.rst doc: change line length limit in contributors guide 2017-06-04 13:18:06 +02:00
index.rst doc: add guidelines on stable and lts releases 2017-02-08 15:33:36 +01:00
patches.rst doc: add author on cc to git fixline alias 2017-08-03 15:27:26 +02:00
stable.rst doc: add guidelines on stable and lts releases 2017-02-08 15:33:36 +01:00
versioning.rst mk: provide option to set major ABI version 2017-03-16 18:20:01 +01:00