-
Notifications
You must be signed in to change notification settings - Fork 449
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
add comment about keeping PRs small in size and simple
- Loading branch information
1 parent
b4dbd0a
commit c718251
Showing
1 changed file
with
6 additions
and
2 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,11 @@ | ||
We thank you in advance for your pull request, and for your interest and support of the Robotics Toolbox for Python. | ||
We thank you in advance for your pull request, and for your interest and support of the Robotics Toolbox for Python. | ||
|
||
You could help us a lot by: | ||
|
||
* Making your pull request relative to the `future` branch. This is where we fix issues and merge pull requests prior to pushing out a new release. | ||
* Including a reference/link to any related issues. | ||
* Providing clear description of the problem you are addressing with the pull request, and the changes proposed . | ||
* Providing a clear description of the problem you are addressing with the pull request, the changes proposed, and their rationale. | ||
* We appreciate code comments explaining what your added block of code does. | ||
* If your PR tackles a number of different issues, please submit multiple smaller/simpler PRs that we can individually accept or not. Otherwise we have to ask you to modify your PR and leave some changes out. Unfortunately GH doesn't let us pick and choose. | ||
* Making your PR as small as possible. Don't include test files, data files, or notebooks that are specific to your project; ensure that notebooks, if of general interest, are saved with output values cleared, and that robot models include only those files required. PyPI has strict size limits on packages, and already RTB is split into a toolbox and data package. | ||
|