-
Notifications
You must be signed in to change notification settings - Fork 159
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
4a045e5
commit 6c33683
Showing
1 changed file
with
12 additions
and
0 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 |
---|---|---|
@@ -0,0 +1,12 @@ | ||
# Pull Requests | ||
|
||
Here are some guidelines for pull requests: | ||
|
||
* All work is submitted via Pull Requests. | ||
* Pull Requests can be submitted as soon as there is code worth discussing. The worst case is that the PR is closed. | ||
* Pull Requests should be made against master | ||
* Pull Requests should be tested, if feasible: | ||
- bugfixes should include regression tests. | ||
- new behavior should at least get minimal exercise. | ||
- new features should include a screenshot | ||
* Don't make 'cleanup' pull requests just to change code style. We don't follow any style guide strictly, and we consider formatting changes unnecessary noise. If you're making functional changes, you can clean up the specific pieces of code you're working on. |