|
1 | | -Thank you so much for your PR! To help us review, please make use of our |
2 | | -[development guide](http://matplotlib.org/devdocs/devel/coding_guide.html#documentation) and fill out the form to the best of your ability. |
| 1 | +Thank you so much for your PR! To help us review, fill out the form to the best of your ability. Also, please make use of the development guide at http://matplotlib.org/devdocs/devel/index.html |
3 | 2 |
|
4 | | -**PR Summary** |
5 | | -- Please use a title that describes the bug being fixed or the feature being implemented-for example "Raises ValueError on Non-Numeric Input to set_xlim" |
6 | | -- Please provide at least 1-2 sentences describing the pull request in detail. |
| 3 | +<!--- Provide a general summary of your changes in the Title above, for example "Raises ValueError on Non-Numeric Input to set_xlim". Please avoid non-descriptive titles such as "Addresses issue #8576"--> |
7 | 4 |
|
8 | | -- What type of PR is this? |
9 | | - - [ ] library bug fix |
10 | | - - [ ] documentation bug fix |
11 | | - - [ ] new documentation |
12 | | - - [ ] new feature |
13 | | - - [ ] other |
| 5 | +## PR Summary |
| 6 | +<!--- Please provide at least 1-2 sentences describing the pull request in detail. Also indicate the type of PR: bug fix or new feature, and documentation or code. --> |
14 | 7 |
|
15 | | -**[PR Checklist](http://matplotlib.org/devel/coding_guide.html#pull-request-checklist)** |
16 | | -- [ ] Code is [tested](http://matplotlib.org/devel/testing.html#testing) using[Py.test](https://docs.pytest.org/en/latest/) style tests) on Py2 and Py3 |
17 | | -- [ ] Code is [PEP 8](https://www.python.org/dev/peps/pep-0008/) compliant |
18 | | -- [ ] New features are documented-and have an example if plot related |
19 | | -- [ ] Documentation is [sphinx](http://matplotlib.org/devdocs/devel/documenting_mpl.html) and [numpydoc](https://github.com/numpy/numpy/blob/master/doc/HOWTO_DOCUMENT.rst.txt) compliant |
20 | | -- [ ] Related issues and pull requests [are listed](https://help.github.com/articles/autolinked-references-and-urls/) |
| 8 | +## Motivation and Context |
| 9 | +<!--- Why is this change required? What problem does it solve? --> |
| 10 | +<!--- If it fixes an open issue, please link to the issue here. --> |
21 | 11 |
|
22 | | -** Major Changes ** |
23 | | -- [ ] major new feature: added an entry to [doc/users/whats_new.rst](../doc/users/whats_new). |
24 | | -- [ ] change the API in a backward-incompatible way: documented it in [doc/api/api_changes.rst](../doc/api/api_changes) |
| 12 | +## PR Checklist |
| 13 | +- [ ] Code is tested using Py.test on Py2 and Py3 |
| 14 | +- [ ] Code is PEP 8 compliant |
| 15 | +- [ ] New features are documented-and have an example if plot related |
| 16 | +- [ ] Documentation is sphinx and numpydoc compliant |
| 17 | +- [ ] major new feature: added an entry to doc/users/whats_new.rst |
| 18 | +- [ ] change the API in a backward-incompatible way: documented it in doc/api/api_changes.rst |
25 | 19 |
|
26 | 20 | We understand that PRs can sometimes feel like a guantlet, especially as the reviews start coming in. Please let us know |
27 | 21 | if the reviews are unclear or the recommended next step is overwhelming, or if you would like help in addressing a reviewer's comments. And please ping us if you've been waiting too long to hear back on your PR! |
0 commit comments