The Spotbugs analysis issues are clouding the view of what's breaking the build. I ran the build locally with `mvn` and `mvn -Dspotbugs.failOnError=false` to verify that the failures on the build are not caused by this change. The Spotbugs errors in the build need more attention and are quite old. I'll put some attention on those ahead of any other PR merges.
On Thu, Jan 5, 2023 at 10:56 AM Gary Gregory <garydgreg...@gmail.com> wrote: > -1 > > Please revert and stop merging PRs that break the build. Run a local build > when you don't know when in doubt. > > Gary > > On Thu, Jan 5, 2023, 10:25 Carl Hall <notificati...@github.com> wrote: > > > Merged #169 <https://github.com/apache/commons-dbutils/pull/169> into > > master. > > > > — > > Reply to this email directly, view it on GitHub > > <https://github.com/apache/commons-dbutils/pull/169#event-8161304510>, > or > > unsubscribe > > < > https://github.com/notifications/unsubscribe-auth/AAJB6N4AX5TJ3ATBC3HCVBTWQ3RVRANCNFSM6AAAAAATQEJEXE > > > > . > > You are receiving this because you are subscribed to this thread.Message > > ID: <apache/commons-dbutils/pull/169/issue_event/8161304...@github.com> > > >