Skip to content

Commit 54d5753

Browse files
ES-AlexanderHwurzburg
authored andcommitted
dev: submitting-patches-back-to-master: recommend using autotests
1 parent 60e6596 commit 54d5753

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

dev/source/docs/submitting-patches-back-to-master.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ You can track the state of your PR from the `Pull Requests list <https://github.
117117
PRs are more likely to be merged quickly if:
118118

119119
- The PR clearly states what changes in behaviour are expected
120-
- Good testing evidence is provided. This could be graphs of logs recorded before and after the change
120+
- Good testing evidence is provided. This could be graphs of logs recorded before and after the change, possibly from a :ref:`simulation <simulation-2>`. Check for existing :ref:`autotests <the-ardupilot-autotest-framework>`, and update them if they get broken by the change. Add new ones to cover new functionality you're introducing, or bug fixes presently not tested, if you are capable.
121121
- Code follows the :ref:`style guide <style-guide>`
122122
- Each commit in the PR affects only one subsystem and the commit title is prefixed with the subsystem name (e.g. "AP_GPS: correct uBlox logging parameter descriptions"). The `Tools/gittools/git-subsystems-split <https://github.com/ArduPilot/ardupilot/blob/master/Tools/gittools/git-subsystems-split>`__ script may be useful for this
123123
- PR passes all automated CI tests. CI tests sometimes fail for reason unrelated to your PR (e.g. dependency failures, "flapping tests"). Please force push the PR to re-run the tests

0 commit comments

Comments
 (0)