]> granicus.if.org Git - php/commitdiff
Mention that zend_parse_parameters should not be tested
authorNikita Popov <nikita.ppv@gmail.com>
Sun, 7 Jul 2019 09:23:58 +0000 (11:23 +0200)
committerNikita Popov <nikita.ppv@gmail.com>
Sun, 7 Jul 2019 09:24:26 +0000 (11:24 +0200)
Also mention that --CREDITS-- section should not be included
anymore. We should also change qa.php.net to stop advertising
these.

[ci skip]

CONTRIBUTING.md

index f97585903238856ead7d9e1e91addf414710f3a2..a084bdea9820bf72cd69240679764151f7eae4aa 100644 (file)
@@ -94,6 +94,14 @@ Submitting test scripts helps us to understand what functionality has changed.
 It is important for the stability and maintainability of PHP that tests are
 comprehensive.
 
+Failure conditions of `zend_parse_parameters`, `ZEND_PARSE_PARAMETERS()` and
+similar functions should not be tested. These parameter parsing APIs are already
+extensively tested, and additional tests only complicate future modifications.
+
+For newly created tests, a `--CREDITS--` section should no longer be included,
+as test authorship is already accurately tracked by Git. If multiple authors
+should be credited, the `Co-authored-by` tag in the commit message may be used.
+
 ## Writing documentation
 
 There are two ways to contribute to the PHP manual. You can edit the manual and