The following commit has been merged in the master branch:
commit a27728daff53309cabfc6dac556aa2645f241116
Author: Benjamin Kaduk <ka...@mit.edu>
Date: Tue Dec 26 09:41:36 2023 -0800
CODING: add code review guidelines
Attempt to enumerate things that reviewers should look for, to ensure
a consistent quality of review. This should be a good start; we can
add to it in the future as we discover more things that we'd like
reviewers to consider.
As a side effect, attempt to enumerate the bits of C99 that we rely on,
past the C89 baseline that we otherwise assume.
Change-Id: I047cc0accc99d2aea7bb7eb1c35ae148bff2ee7d
Reviewed-on: https://gerrit.openafs.org/15611
Reviewed-by: Michael Meffie <mmef...@sinenomine.net>
Reviewed-by: Andrew Deason <adea...@sinenomine.net>
Reviewed-by: Mark Vitale <mvit...@sinenomine.net>
Reviewed-by: Cheyenne Wills <cwi...@sinenomine.net>
Reviewed-by: Benjamin Kaduk <ka...@mit.edu>
Tested-by: Benjamin Kaduk <ka...@mit.edu>
CODING | 190 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 190 insertions(+)
--
OpenAFS Master Repository
_______________________________________________
OpenAFS-cvs mailing list
OpenAFS-cvs@openafs.org
https://lists.openafs.org/mailman/listinfo/openafs-cvs
