forked from boto/boto
-
Notifications
You must be signed in to change notification settings - Fork 0
/
CONTRIBUTING
92 lines (69 loc) · 3.16 KB
/
CONTRIBUTING
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
============
Contributing
============
For more information, please see the official contribution docs at
http://docs.pythonboto.org/en/latest/contributing.html.
Contributing Code
=================
* A good patch:
* is clear.
* works across all supported versions of Python.
* follows the existing style of the code base (PEP-8).
* has comments included as needed.
* A test case that demonstrates the previous flaw that now passes
with the included patch.
* If it adds/changes a public API, it must also include documentation
for those changes.
* Must be appropriately licensed (New BSD).
Reporting An Issue/Feature
==========================
* Check to see if there's an existing issue/pull request for the
bug/feature. All issues are at https://github.com/boto/boto/issues
and pull reqs are at https://github.com/boto/boto/pulls.
* If there isn't an existing issue there, please file an issue. The ideal
report includes:
* A description of the problem/suggestion.
* How to recreate the bug.
* If relevant, including the versions of your:
* Python interpreter
* boto
* Optionally of the other dependencies involved
* If possible, create a pull request with a (failing) test case demonstrating
what's wrong. This makes the process for fixing bugs quicker & gets issues
resolved sooner.
Maintenance
===========
You do not have to be labeled as a
"maintainer" to be able to help with the triaging, resolving, and
reviewing of boto issues and pull requests.
These are the processes that the maintainers follow, that you can also
follow to help speed up the resolution of an issue or pull request:
Pull Requests
-------------
There are some key points that are needed to be met before a pull request
can be merged:
* All tests must pass for all python versions.
* All pull requests require tests that either test the new feature or test
that the specific bug is fixed. Pull requests for minor things like
adding a new region or fixing a typo do not need tests.
* Must follow PEP8 conventions.
* All changes must be backwards compatible.
* If the pull request is for a feature, make sure a link to the corresponding
API is provided in order to easily check that the feature matches up
with the service's API.
The best way to help with pull requests is to comment on pull requests by
noting if any of these key points are missing, it will both help get feedback
sooner to the issuer of the pull request and make it easier to determine for
an individual with write permissions to the repository if a pull request
is ready to be merged.
Issues
------
Here are the best ways to help with open issues:
* If there is an issue without a set of instructions on how to reproduce the
bug, feel free to try to reproduce the issue, comment with the minimal
amount of steps to reproduce the bug (a code snippet would be ideal). If
there is not a set of steps that can be made to reproduce the issue, at
least make sure there are debug logs that capture the unexpected behavior.
* Consolidate all related issue to one issues by closing out related issues
and linking them to the single issue that outlines the general issue.
* Submit pull requests for open issues.