diff --git a/Patches.md b/Patches.md index e97f448..a99729f 100644 --- a/Patches.md +++ b/Patches.md @@ -2,7 +2,7 @@ Please create a new ticket to track the issue unless your patch addresses an issue that is already ticketed. If you're not sure whether there is a ticket for your issue, just make a new one. Someone will point you in the right direction if your ticket is a duplicate. -We use git as our versoin control system and the tahoe source code is hosted on [Github](https://github.com/tahoe-lafs/tahoe-lafs). Github requires that you have a Github account in order to submit patches, so you will have to create one if you don't already have one. If you are unfamiliar with Github or git we recommend the following resources: +We use git as our version control system and the tahoe source code is hosted on [Github](https://github.com/tahoe-lafs/tahoe-lafs). Github requires that you have a Github account in order to submit patches, so you will have to create one if you don't already have one. If you are unfamiliar with Github or git we recommend the following resources: * [Github - Set up git](https://help.github.com/articles/set-up-git) * [Github - Fork a repo](https://help.github.com/articles/fork-a-repo) * [git-scm.com/book - A great overall git resource](http://git-scm.com/book) @@ -15,13 +15,13 @@ Navigate to the `tahoe-lafs` folder and type `git branch` to make sure `master` Now, hack at Tahoe-LAFS! -Once you think you have solved the issue run the test suite with `python setup.py trial`. This ensures that you didn't accidentally break something while working on your patch. +Once you think you have solved the issue run the test suite with `python setup.py test`. This ensures that you didn't accidentally break something while working on your patch. -Now that all of the existing unit tests pass, [write your own tests](https://tahoe-lafs.org/trac/tahoe-lafs/wiki/HowToWriteTests) for the ticket if they don't already exist. +Now that all of the existing unit tests pass, [write your own tests](https://tahoe-lafs.org/trac/tahoe-lafs/wiki/HowToWriteTests) for the ticket if they don't already exist, and make sure that they pass. Also ensure that `make pyflakes` finds no errors or warnings. Finally, push your branch to Github and [open a pull request](https://help.github.com/articles/creating-a-pull-request). Please add the tag `review-needed` to the ticket and add a link to the pull request. If you had trouble writing unit tests for your patch, don't worry. Submit the pull request anyway and add the tag `tests-needed` to the ticket. # Design reviews -To request a design review, explain your design (with or without a patch) and then add the [design-review-needed]query:keywords~=design-review-needed&status=!closed&group=milestone tag. Completion of a design review will normally not directly result in a patch being committed. The main goal of a design review is to give the person working on the ticket confidence that there are no show-stopping issues with the approach they are taking, and to get feedback on smaller issues that are useful to take into account before doing further work on a patch. \ No newline at end of file +To request a design review, just explain your design (with or without a patch) and ask for feedback. Completion of a design review will normally not directly result in a patch being committed. The main goal of a design review is to give the person working on the ticket confidence that there are no show-stopping issues with the approach they are taking, and to get feedback on smaller issues that are useful to take into account before doing further work on a patch. \ No newline at end of file