cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joe Bowser <bows...@gmail.com>
Subject We need better contributions from new people
Date Wed, 12 Jun 2013 06:09:00 GMT
Hey

I decided to check my e-mail and respond to an issue.  It turns out
that we're not properly testing or even checking the pull requests
that we're getting into the project. The bug in question is CB-3766,
which was caused by a patch accepted to fix CB-2458.  The error isn't
totally obvious, and doesn't get easily picked up on unit test (our
CordovaWebView test mostly works, but fires a TIMING ERROR), but if
you use an emulator, due to the emulator's crappiness, it apparently
creates an ugly stack trace.

Honestly, there's no way that this should have made it in.  I
personally think that I'm going to have to rename loadUrlIntoView to
initAndLoadUrl or something more obvious, since it's too similar to
loadUrl.  That being said, we need to be more strict about quality
while at the same time encouraging people to contribute.  I'd rather
deal with hundreds of pull requests than hundreds of issues where
people know the answer but don't tell you.

Any ideas how we can accomplish this?  Has anyone else seen pull
requests get in that shouldn't have?

Joe

Mime
View raw message