A RetroSearch Logo

Home - News ( United States | United Kingdom | Italy | Germany ) - Football scores

Search Query:

Showing content from https://mail.python.org/pipermail/python-dev/2010-April/099811.html below:

[Python-Dev] Anyone can do patch reviews

[Python-Dev] Anyone can do patch reviewsNick Coghlan ncoghlan at gmail.com
Tue Apr 27 23:47:52 CEST 2010
Ezio Melotti wrote:
> When I'm writing the patch it's usually easy, I write the tests, see
> that they fail, write the fix, see that they pass.
> When I'm reviewing the patch, I apply the patch, see that the tests
> pass, svn revert the fix, check that they fail.
> Most of the patches affect just a couple of files, so applying the whole
> patch and then revert is usually trivial and probably easier than having
> to deal with two separate files for patch and tests.

This would be pretty close to my typical workflow as well (*looks at
list of assigned bugs that hasn't moved in weeks* well, it is the
workflow when I actually *do* some Python coding...)

Cheers,
Nick.

-- 
Nick Coghlan   |   ncoghlan at gmail.com   |   Brisbane, Australia
---------------------------------------------------------------
More information about the Python-Dev mailing list

RetroSearch is an open source project built by @garambo | Open a GitHub Issue

Search and Browse the WWW like it's 1997 | Search results from DuckDuckGo

HTML: 3.2 | Encoding: UTF-8 | Version: 0.7.4