You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by Gary Gregory <ga...@gmail.com> on 2022/12/09 11:50:37 UTC
[Dbutils] PR merges build
-1 to the commit or PR that broke the build. To whomever merged it, please
revert.
Before you merge a PR, if you are uncertain that it is safe, test it
locally.
Gary
Re: [Dbutils] PR merges build
Posted by Gary Gregory <ga...@gmail.com>.
Thank you Carl!
Gary
On Mon, Jan 9, 2023, 23:03 Carl Hall <ca...@gmail.com> wrote:
> The fix was thankfully straightforward and quickly evident. Spotbugs was
> rolled back to a previous version (4.2.3). The analysis bugs raised by the
> upgrade will be addressed in a branch away from master before upgrading.
> Builds are green again.
>
> On Mon, Jan 9, 2023 at 9:48 PM Carl Hall <ca...@gmail.com> wrote:
>
> > Apologies for causing the build breakage, and missing your message. This
> > is my project priority until resolved. Thanks for staying on top of this.
> >
> > On Fri, Dec 16, 2022 at 4:11 AM Gary Gregory <ga...@gmail.com>
> > wrote:
> >
> >> Ping.
> >>
> >> Gary
> >>
> >> On Fri, Dec 9, 2022, 06:50 Gary Gregory <ga...@gmail.com> wrote:
> >>
> >> > -1 to the commit or PR that broke the build. To whomever merged it,
> >> please
> >> > revert.
> >> >
> >> > Before you merge a PR, if you are uncertain that it is safe, test it
> >> > locally.
> >> >
> >> > Gary
> >> >
> >>
> >
>
Re: [Dbutils] PR merges build
Posted by Carl Hall <ca...@gmail.com>.
The fix was thankfully straightforward and quickly evident. Spotbugs was
rolled back to a previous version (4.2.3). The analysis bugs raised by the
upgrade will be addressed in a branch away from master before upgrading.
Builds are green again.
On Mon, Jan 9, 2023 at 9:48 PM Carl Hall <ca...@gmail.com> wrote:
> Apologies for causing the build breakage, and missing your message. This
> is my project priority until resolved. Thanks for staying on top of this.
>
> On Fri, Dec 16, 2022 at 4:11 AM Gary Gregory <ga...@gmail.com>
> wrote:
>
>> Ping.
>>
>> Gary
>>
>> On Fri, Dec 9, 2022, 06:50 Gary Gregory <ga...@gmail.com> wrote:
>>
>> > -1 to the commit or PR that broke the build. To whomever merged it,
>> please
>> > revert.
>> >
>> > Before you merge a PR, if you are uncertain that it is safe, test it
>> > locally.
>> >
>> > Gary
>> >
>>
>
Re: [Dbutils] PR merges build
Posted by Carl Hall <ca...@gmail.com>.
Apologies for causing the build breakage, and missing your message. This is
my project priority until resolved. Thanks for staying on top of this.
On Fri, Dec 16, 2022 at 4:11 AM Gary Gregory <ga...@gmail.com> wrote:
> Ping.
>
> Gary
>
> On Fri, Dec 9, 2022, 06:50 Gary Gregory <ga...@gmail.com> wrote:
>
> > -1 to the commit or PR that broke the build. To whomever merged it,
> please
> > revert.
> >
> > Before you merge a PR, if you are uncertain that it is safe, test it
> > locally.
> >
> > Gary
> >
>
Re: [Dbutils] PR merges build
Posted by Gary Gregory <ga...@gmail.com>.
Ping.
Gary
On Fri, Dec 9, 2022, 06:50 Gary Gregory <ga...@gmail.com> wrote:
> -1 to the commit or PR that broke the build. To whomever merged it, please
> revert.
>
> Before you merge a PR, if you are uncertain that it is safe, test it
> locally.
>
> Gary
>