New PR sent: https://github.com/apache/activemq-artemis/pull/3316
On Mon, Oct 26, 2020 at 9:33 PM Clebert Suconic <[email protected]> wrote: > > I will rework the docs tomorrow, close the PR and send a fresh one at > the end of tomorrow. > > On Mon, Oct 26, 2020 at 12:58 PM Clebert Suconic > <[email protected]> wrote: > > > > I thought you would be able to unresolve a conversation... > > > > So, if you could please either open a new conversation, or wait the new PR. > > > > I can't manage 200+ github comments without resolving conversations > > after I deal with the conversation. (my browser actually crashed one > > time due to the size). > > > > On Mon, Oct 26, 2020 at 6:54 AM Robbie Gemmell <[email protected]> > > wrote: > > > > > > On Fri, 23 Oct 2020 at 21:47, Clebert Suconic <[email protected]> > > > wrote: > > > > > > > > I was thinking the same actually... I > > > > > > > > I will just resolve a few last comments, and I will then clone it and > > > > open a new one... > > > > > > > > > > > > just one request for now at least.. if you make a comment on a > > > > previously resolved conversation, please unresolve it.. as I'm not > > > > being able to look at resolved conversations. > > > > > > > > > > I actually can't unresolve the conversations, even though I made most > > > of them, I had just commented to someone else about that earlier on > > > Friday. > > > > > > > As of now I finished SSL implementation and I will polish things on > > > > monday (do a set of review from myself and open a new PR). > > > > > > > > Have a nice weekend! > > > > > > > > > > > > On Fri, Oct 23, 2020 at 1:00 PM Robbie Gemmell > > > > <[email protected]> wrote: > > > > > > > > > > It might be worth raising a fresh PR for finishing off review after > > > > > you've done the next set of polish, the existing one has seen such a > > > > > bunch of change and as a result has got so big as to be a bit > > > > > unmanageable :) > > > > > > > > > > On Fri, 23 Oct 2020 at 15:49, Clebert Suconic > > > > > <[email protected]> wrote: > > > > > > > > > > > > I am obviously pushing back the release.. I will do as soon as I we > > > > > > merge https://github.com/apache/activemq-artemis/pull/3294 (which > > > > > > will > > > > > > include some tests before I merge it) > > > > > > > > > > > > On Mon, Oct 19, 2020 at 10:47 AM Clebert Suconic > > > > > > <[email protected]> wrote: > > > > > > > > > > > > > > I'm planning to finish my work on Server Connections > > > > > > > (https://github.com/apache/activemq-artemis/pull/3294) on the > > > > > > > next day > > > > > > > or 2, and I would like to start cutting a release on Friday, with > > > > > > > the > > > > > > > Vote thread sent on monday. > > > > > > > > > > > > > > That means I will have everything ready to go for a vote on > > > > > > > Friday, > > > > > > > and the release vote to be sent on monday. > > > > > > > > > > > > > > for the github Pull Requests under review I will also make sure we > > > > > > > merge current relevant PRs... so, if you could make sure you send > > > > > > > your > > > > > > > pending changes.. and could even nudge me with a comment on your > > > > > > > github pull requet tagging my name that you would like to include > > > > > > > such > > > > > > > PR. (feel free to communicate here as well if you prefer). > > > > > > > > > > > > > > -- > > > > > > > Clebert Suconic > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > Clebert Suconic > > > > > > > > > > > > > > > > -- > > > > Clebert Suconic > > > > > > > > -- > > Clebert Suconic > > > > -- > Clebert Suconic -- Clebert Suconic
