Re: Bug day report

Roman Joost <[email protected]>
Newsgroups gmane.comp.web.zope.devel
Organization BroMeCo
Message-ID <[email protected]>
Hi Tres,

On Thu, Jun 17, 2010 at 07:59:18AM -0400, Tres Seaver wrote:
> Roman, I landed my version of the fix in 'zope.testrunner', but didn't
> touch it in 'zope.testing', since you were working on it.  I was rushing
> a little because I wanted to release 'zope.testrunner', but wanted to
> check with you first.  The patch is here:
> 
> http://svn.zope.org/zope.testrunner/trunk/src/zope/testrunner/find.py?rev=113540&view=diff&r1=113540&r2=113539&p1=zope.testrunner/trunk/src/zope/testrunner/find.py&p2=/zope.testrunner/trunk/src/zope/testrunner/find.py
> 
> I'm fine with letting you land it in zope.testing, or even tweak it on
> the zope.testruner side, if you see a better fix.
Thank you. I actually had troubles running all tests in zope.testrunner.
I need to check why to be sure, to avoid commiting something broken.

I usually prefer to let someone more experienced review my code,
although it seems to be not worth it for this little fix.

So to clarify - can I commit the patch to zope.testrunner after I
verified that the tests are working fine on my local sandbox?

Cheers,
-- 
Roman Joost
www: http://www.romanofski.de
email: [email protected]

_______________________________________________
Zope-Dev maillist  -  [email protected]
https://mail.zope.org/mailman/listinfo/zope-dev
**  No cross posts or HTML encoding!  **
(Related lists - 
 https://mail.zope.org/mailman/listinfo/zope-announce
 https://mail.zope.org/mailman/listinfo/zope )
signature.asc (application/pgp-signature, 190 B)
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)

iD8DBQFMGw+w8CscaEkpx0IRAuhdAJ9cVXbtNTreGnLNvbRiMDhvw5TlOgCeJACw
ICtxPY32FIboTXEl0tRfKok=
=grHE
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.