Bump! Please vote on the release On Thu, Sep 8, 2016 at 9:06 AM, Steven Gill <stevengil...@gmail.com> wrote:
> I can review and merge > > On Sep 8, 2016 9:00 AM, "David Barth" <david.ba...@canonical.com> wrote: > >> Hey guys, >> >> Alex has a set of fixes for ubuntu code in the following plugins: >> >> - https://github.com/apache/cordova-plugin-inappbrowser/pull/174 >> - https://github.com/apache/cordova-plugin-battery-status/pull/38 >> - https://github.com/cordova-ubuntu/cordova-plugin-geolocation/pull/2 >> >> Should we get another review from plugin maintainers or should i merge to >> get that in time for the release ? >> >> David >> >> >> >> On Wed, Sep 7, 2016 at 4:47 PM, julio cesar sanchez < >> jcesarmob...@gmail.com> >> wrote: >> >> > Or maybe we should wait for the corodvaDependencies update? I think >> there >> > are PRs for all the plugins >> > >> > 2016-09-07 16:21 GMT+02:00 Steven Gill <stevengil...@gmail.com>: >> > >> > > Yes. I'll move forward with it >> > > >> > > On Sep 7, 2016 6:25 AM, "julio cesar sanchez" <jcesarmob...@gmail.com >> > >> > > wrote: >> > > >> > > > Issues won't stop coming, should we do the release this week? >> > > > >> > > > 2016-08-29 13:26 GMT+02:00 Kerri Shotts <kerrisho...@gmail.com>: >> > > > >> > > > > I've got iPads that support multitasking, so I'll take a look. >> > > > > >> > > > > ~ Kerri >> > > > > >> > > > > >> > > > > >> > > > > On Mon, Aug 29, 2016 at 5:57 AM -0500, "julio cesar sanchez" < >> > > > > jcesarmob...@gmail.com> wrote: >> > > > > >> > > > > >> > > > > >> > > > > >> > > > > >> > > > > >> > > > > >> > > > > >> > > > > >> > > > > >> > > > > If somebody with an iPad that supports multitasking can test this >> > PR... >> > > > > https://github.com/apache/cordova-plugin-statusbar/pull/62 >> > > > > >> > > > > I have reviewed the code and looks good, bud I don't have an iPad >> > that >> > > > > supports multitasking to test it, and I don't trust the simulator >> in >> > > this >> > > > > case as I tried to fix that issue in a different way, got it >> working >> > on >> > > > the >> > > > > simulator but the reporter said that it didn't work as expected >> on a >> > > real >> > > > > device. >> > > > > >> > > > > 2016-08-25 0:54 GMT+02:00 julio cesar sanchez : >> > > > > >> > > > > > PRs to review >> > > > > > https://github.com/apache/cordova-plugin-contacts/pull/131 >> > > > > > https://github.com/apache/cordova-plugin-camera/pull/229 >> > > > > > >> > > > > > 2016-08-24 20:20 GMT+02:00 Steven Gill : >> > > > > > >> > > > > >> Hey Scott, >> > > > > >> >> > > > > >> Thanks for pointing out the PR. I'll take a look. >> > > > > >> >> > > > > >> The reality of the situation is that cordova has many repos and >> > not >> > > > > enough >> > > > > >> committers currently. This leads to PRs sitting unreviewed. >> Plugin >> > > PRs >> > > > > are >> > > > > >> unique in that they can involve various different native >> language >> > > > > changes. >> > > > > >> This could require more than 1 committer needing to review. For >> > > > example, >> > > > > >> some committers are more experienced with android vs ios, so >> they >> > > > won't >> > > > > be >> > > > > >> able to review the ios portion of the plugin PR. Another issue >> > with >> > > > > plugin >> > > > > >> PRs is that they may try to add a new feature (without >> discussion) >> > > and >> > > > > may >> > > > > >> not add similar functionality for other platforms that support >> the >> > > > > plugin. >> > > > > >> And of course lots of PRs don't include tests. >> > > > > >> >> > > > > >> The name being attached to component doesn't mean anything. >> Issues >> > > by >> > > > > >> default go unassigned. >> > > > > >> >> > > > > >> I can only speak on behalf of the regular Adobe cordova >> commiters >> > (4 >> > > > of >> > > > > >> us). Our focus is usually spent on platforms + tools fixes and >> > > > > >> improvements. >> > > > > >> >> > > > > >> The best way to get attention for your PR is to post it to >> slack + >> > > > > here. I >> > > > > >> usually do a review of PRs when I do a plugins release, but I >> > won't >> > > be >> > > > > >> able >> > > > > >> to go through them all. This is why I ask for people to bring >> some >> > > to >> > > > my >> > > > > >> attention during releases. >> > > > > >> >> > > > > >> Lastly, we would love to help onboard new committers to help >> > manage >> > > > > >> plugins. If you are interested in such a role, LMK! >> > > > > >> >> > > > > >> Cheers, >> > > > > >> -Steve >> > > > > >> >> > > > > >> On Wed, Aug 24, 2016 at 6:14 AM, Scott Bradshaw >> > > > > >> wrote: >> > > > > >> >> > > > > >> > Hi Steven - Can you look at camera PR 197 >> > > > > >> > ? This >> > > > > >> addresses >> > > > > >> > a major bug in the Android camera plugin where it won't >> rotate >> > > > > pictures >> > > > > >> > chosen from the library. This bug has been around since >> Android >> > > 4.4 >> > > > > was >> > > > > >> > released. I worked on this at the beginning of the year, but >> > it's >> > > > > just >> > > > > >> > sitting there. I'm new to Cordova, so I don't know exactly >> what >> > > the >> > > > > >> > process is to get someone to look at it. I've joined Slack to >> > find >> > > > > >> someone >> > > > > >> > to look at it, but no luck. Your name appears to be a >> attached >> > > > > >> > > > com.atlassian.jira.jira-projec >> ts-plugin:components-panel> >> > > > > >> > to most of the plugins, including camera. This also isn't the >> > only >> > > > PR >> > > > > >> out >> > > > > >> > there for camera. Several are just sitting there waiting for >> > > review. >> > > > > At >> > > > > >> > what point do they get reviewed? >> > > > > >> > >> > > > > >> > On Wed, Aug 24, 2016 at 1:16 AM, Steven Gill >> > > > > >> > wrote: >> > > > > >> > >> > > > > >> > > Thoughts? Any PRs to bring to my attention? >> > > > > >> > > >> > > > > >> > > Will aim to start it later this week. >> > > > > >> > > >> > > > > >> > >> > > > > >> >> > > > > > >> > > > > > >> > > > > >> > > > > >> > > > > >> > > > > >> > > > > >> > > > > >> > > > >> > > >> > >> >