I think all of them are now merged, except for https://github.com/apache/tomee/pull/370 <https://github.com/apache/tomee/pull/370>
My concern here is we are changing the equals implementation, which is usually auto generated by our IDE’s. So, most likely these changes will be lost if someone regenerates the equals method, unless their IDE is set to use Objects.equals. This is no blocker, but I would like to discuss it further before moving forward with the PR. Maybe we should force all .equals to use Objects.equals? With a checkstyle check (not sure if they have a check for that)? Cheers, Roberto > On 28 Jan 2019, at 15:30, Bruno Baptista <bruno...@gmail.com> wrote: > > Thanks Roberto. > > Bruno Baptista > https://twitter.com/brunobat_ > > > On 28/01/19 15:29, Roberto Cortez wrote: >> I’ll be looking into the current list of PR’s and try to merge the ones that >> are ready. >> >> Cheers, >> Roberto >> >>> On 24 Jan 2019, at 15:23, Otávio Gonçalves de Santana >>> <osant...@tomitribe.com> wrote: >>> >>> Thank you for the explanation it Roberto. >>> >>> On Thu, Jan 24, 2019 at 12:50 PM Roberto Cortez >>> <radcor...@yahoo.com.invalid> >>> wrote: >>> >>>> We didn’t. >>>> >>>> If we are moving forward with the release, I would prefer to keep these in >>>> stand by until we are done. Right now, we are on a green build, and we >>>> can’t be sure if any of these PR’s would break it, so better to be safe :) >>>> >>>>> On 24 Jan 2019, at 12:05, Otávio Gonçalves de Santana < >>>> osant...@tomitribe.com> wrote: >>>>> Hello everyone, please don't forget these PRs >>>>> >>>>> On Wed, Jan 23, 2019 at 8:01 AM Bruno Baptista <bruno...@gmail.com> >>>> wrote: >>>>>> Hi, >>>>>> >>>>>> Can one of the committers please take a look at these PRs? >>>>>> >>>>>> https://github.com/apache/tomee/pull/377 >>>>>> https://github.com/apache/tomee/pull/376 >>>>>> https://github.com/apache/tomee/pull/375 >>>>>> https://github.com/apache/tomee/pull/370 >>>>>> https://github.com/apache/tomee/pull/366 >>>>>> https://github.com/apache/tomee/pull/363 >>>>>> https://github.com/apache/tomee/pull/362 >>>>>> https://github.com/apache/tomee/pull/361 >>>>>> >>>>>> Cheers >>>>>> >>>>>> -- >>>>>> Bruno Baptista >>>>>> https://twitter.com/brunobat_ >>>>>> >>>>>> >>>>>> >>>>
